Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Add the Packet provider #3914

Closed
wants to merge 11 commits into from
Closed

Conversation

displague
Copy link
Contributor

@displague displague commented Jul 17, 2020

I'm drafting this PR early to facilitate conversation and to add context to any provider creation questions that come up in the process.

I've included #1112 from @smarterclayton in this PR with the intention of addressing PR feedback fro #1112 while adding my own observations.

I am happy to rebase and squash away the messy commits that will make their way into this PR along the way.


Maintainers, it's not clear to me what todo items I should aim for here. Feel free to edit this PR description to include appropriate bullets for a Platform integration

This PR:

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jul 17, 2020
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign abhinavdahiya
You can assign the PR to them by writing /assign @abhinavdahiya in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jul 17, 2020
@openshift-ci-robot
Copy link
Contributor

Hi @displague. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

displague and others added 6 commits July 27, 2020 22:49
Signed-off-by: Marques Johansson <[email protected]>
This covers the minimal steps and process to go from "nothing" to
"OpenShift is fully capable of running on your platform". Heavily
work in progress, but should capture the why, our support levels,
and our target config, as well as mechanical steps to get down the
line.
@openshift-ci-robot openshift-ci-robot added do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jul 28, 2020
@openshift-ci-robot
Copy link
Contributor

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the openshift org. You can then trigger verification by writing /verify-owners in a comment.

  • displague
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.

@openshift-ci-robot
Copy link
Contributor

@displague: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 30, 2020
@russellb
Copy link
Member

russellb commented Aug 12, 2020

Hi @displague ! It would be great to start by writing up an enhancement here: https://github.com/openshift/enhancements

I think a design document would be appropriate to discuss the various important decision points about how the packet platform will be integrated. Those questions span more than just the installer repository. There's a document we wrote that highlighted some of the areas we expect to discuss for packet. Hopefully you have it?

Thanks!

@displague
Copy link
Contributor Author

Hi @russellb! I have received a few documents about this and we broadly have approaches in mind for the expectations.

I'd be happy to start by opening an enhancements issue, but I don't appear to have access to that repository (404).

@russellb
Copy link
Member

Hi @russellb! I have received a few documents about this and we broadly have approaches in mind for the expectations.

I'd be happy to start by opening an enhancements issue, but I don't appear to have access to that repository (404).

sorry, I had a typo in the repo name. Give it another try.

@displague
Copy link
Contributor Author

@russellb I see it now (typo). What makes the most sense for this?

A single document here:
https://github.com/openshift/enhancements/tree/master/enhancements/installer

This is what I generally see for providers. BareMetal documents appear in a few other locations, but I'm not sure if I will need to explore additional enhancement areas outside of the provider scope.

@russellb
Copy link
Member

@russellb I see it now (typo). What makes the most sense for this?

A single document here:
https://github.com/openshift/enhancements/tree/master/enhancements/installer

This is what I generally see for providers. BareMetal documents appear in a few other locations, but I'm not sure if I will need to explore additional enhancement areas outside of the provider scope.

This repo didn't always exist, so there may not be a perfect example to follow. I'd create a document just in the root enhancements directory since it applies to more than a single category. For example, it probably touches on things related to installer, network, machine-config, machine-api, rhcos, and storage`. Don't worry about getting it perfect. I'm just hoping we can touch on all the important design points before you have to write all of the code.

@displague displague mentioned this pull request Dec 10, 2020
6 tasks
@displague
Copy link
Contributor Author

Moving to #4472

@displague displague closed this Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants