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
HOSTEDCP-1429: hypershift-operator: add a controller for t-shirt sizing #3686
HOSTEDCP-1429: hypershift-operator: add a controller for t-shirt sizing #3686
Conversation
@stevekuznetsov: This pull request references HOSTEDCP-1429 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
✅ Deploy Preview for hypershift-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Looks like I need to write that integration test :D |
35c2fba
to
e5626c5
Compare
e5626c5
to
31df990
Compare
|
5c4377f
to
4ff8a15
Compare
@csrwng integration test:
|
4ff8a15
to
01c4be5
Compare
01c4be5
to
08c9faf
Compare
08c9faf
to
3cca91b
Compare
There are only two places in k8s where ordering matters for what you're applying to the server: namespaces and CRDs. When we declare that we need CRDs, we also need to know that they are established before they can be used. New options to the install command allow us to wait on this explicitly, instead of implcitly, and output separate files during render to allow third parties to do so as well. Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
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.
nits
3cca91b
to
ed0ab72
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: csrwng, stevekuznetsov 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 |
hypershift-operator/controllers/hostedclustersizing/hostedclustersizing_controller.go
Outdated
Show resolved
Hide resolved
ed0ab72
to
451f123
Compare
/hold |
451f123
to
25fc012
Compare
/retest |
/hold cancel |
@stevekuznetsov one thing that occurred to me is that we should skip reconcile of a hostedcluster that either has a deletionTimestamp or is paused. Other than that, lgtm |
25fc012
to
341d4c6
Compare
hypershift-operator/controllers/hostedclustersizing/hostedclustersizing_controller.go
Outdated
Show resolved
Hide resolved
341d4c6
to
d6f9feb
Compare
/lgtm |
Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
d6f9feb
to
c17b642
Compare
/lgtm |
@stevekuznetsov: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
be38396
into
openshift:main
/assign @csrwng