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

[KONFLUX-179]: ADR for provisioning clusters #172

Merged
merged 6 commits into from
Jun 5, 2024

Conversation

amisstea
Copy link
Contributor

@amisstea amisstea commented Mar 7, 2024

This dictates how clusters will be provisioned from within an integration pipeline using the Cluster as a Service Operator.

@amisstea amisstea requested a review from a team as a code owner March 7, 2024 16:28
@amisstea
Copy link
Contributor Author

amisstea commented Mar 7, 2024

This ADR replaces #168.

@ralphbean
Copy link
Member

@dperaza4dustbit, @gbenhaim I'd love to see your review comments to this proposal.

Copy link
Member

@arewm arewm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for writing this up, @amisstea!

ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
ADR/0033-provisioning-test-resources.md Outdated Show resolved Hide resolved
This dictates how clusters will be provisioned from within an integration
pipeline using the Cluster as a Service Operator.

Signed-off-by: Alex Misstear <[email protected]>
amisstea and others added 2 commits May 20, 2024 16:22
Superfluous OIDC config details are also removed.

Signed-off-by: Alex Misstear <[email protected]>
@amisstea
Copy link
Contributor Author

@arewm could you please give this another review pass?

@amisstea amisstea requested a review from arewm May 29, 2024 20:40
Remove some uneccesary parenthesis as well.

Signed-off-by: Alex Misstear <[email protected]>
Copy link

@dperaza4dustbit dperaza4dustbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, there will be no need for changes to KubeSaw as tiers are being decoupled from Host Operator and can be managed independent in Infra deployment or what ever mechanism we use upstream for Konflux install/upate

@amisstea amisstea merged commit cd41772 into konflux-ci:main Jun 5, 2024
1 check passed
@amisstea amisstea deleted the KONFLUX-179-caas branch June 5, 2024 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feat Something new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants