-
Notifications
You must be signed in to change notification settings - Fork 398
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
Refine tier 3 platform support definition #1444
Conversation
Updates tier 3 platform support definition to require demonstrated ability to target the platform and documentation available for doing so. Signed-off-by: hasheddan <georgedanielmangum@gmail.com>
|
||
- Official builds are not available. | ||
- Automated testing is not setup. | ||
- Documentation on how to build for the platform is available. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am trying to decide if this documentation belongs in sig-release/release-engineering or in https://github.com/kubernetes/community/tree/master/contributors/devel/sig-architecture. I would like to keep as much of this information located together as possible, but we already have the beginning of guide in k/community. Ultimately, it seems like this primarily falls under release-engineering in terms of shepherding people through the process, and we also need to make sure release-engineering has knowledge of how to build all platforms, so I am somewhat inclined to move the guide here, point folks to it, and have Tier 3 build docs also live here.
Thoughts? @dims @saschagrunert
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
release-engineering sounds good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree this should live here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
|
||
- Official builds are not available. | ||
- Automated testing is not setup. | ||
- Documentation on how to build for the platform is available. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree this should live here.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hasheddan, saschagrunert The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this:
/kind documentation
What this PR does / why we need it:
Updates tier 3 platform support definition to require demonstrated
ability to target the platform and documentation available for doing so.
Signed-off-by: hasheddan georgedanielmangum@gmail.com
Which issue(s) this PR fixes:
xref #1337
Special notes for your reviewer:
Updates based on feedback in SIG Arch meeting.
/assign @saschagrunert @justaugustus
/cc @dims