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
MGMT-16521: Identify Operator-based Deployment Data #5906
MGMT-16521: Identify Operator-based Deployment Data #5906
Conversation
@CrystalChun: This pull request references MGMT-16521 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 sub-task 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. |
Skipping CI for Draft Pull Request. |
@CrystalChun: This pull request references MGMT-16521 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 sub-task 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. |
9c62fa8
to
08c584c
Compare
@CrystalChun: This pull request references MGMT-16521 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 sub-task 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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5906 +/- ##
==========================================
+ Coverage 68.10% 68.59% +0.48%
==========================================
Files 235 236 +1
Lines 34642 35382 +740
==========================================
+ Hits 23594 24271 +677
- Misses 8986 9020 +34
- Partials 2062 2091 +29
|
internal/controller/controllers/agentserviceconfig_controller.go
Outdated
Show resolved
Hide resolved
08c584c
to
0ac172b
Compare
internal/controller/controllers/agentserviceconfig_controller.go
Outdated
Show resolved
Hide resolved
0ac172b
to
d4aed6d
Compare
https://issues.redhat.com/browse/MGMT-16521 Read the deployment data from the environment variables when deploying with the operator. Possible values include ACM, MCE, and operator (indicates stand-alone operator). The block querying for ACM and MCE deployments are a temporary work-around until ACM/MCE allows injecting environment variables into the infrastructure operator deployment.
d4aed6d
to
7e16fea
Compare
@CrystalChun: This pull request references MGMT-16521 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 sub-task 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. |
/lgtm |
/hold |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CrystalChun, rccrdpccl 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 |
/unhold |
@CrystalChun: all tests passed! 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. |
[ART PR BUILD NOTIFIER] This PR has been included in build ose-agent-installer-api-server-container-v4.16.0-202401311150.p0.g89f0a6b.assembly.stream for distgit ose-agent-installer-api-server. |
https://issues.redhat.com/browse/MGMT-16521 Read the deployment data from the environment variables when deploying with the operator. Possible values include ACM, MCE, and operator (indicates stand-alone operator). The block querying for ACM and MCE deployments are a temporary work-around until ACM/MCE allows injecting environment variables into the infrastructure operator deployment.
/cherry-pick release-ocm-2.9 |
@CrystalChun: new pull request created: #5989 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 kubernetes/test-infra repository. |
https://issues.redhat.com/browse/MGMT-16521
Read the deployment data from the environment
variables when deploying with the operator.
Possible values include ACM, MCE, and operator
(indicates stand-alone operator).
The block querying for ACM and MCE deployments
are a temporary work-around until ACM/MCE
allows injecting environment variables into the
infrastructure operator deployment.
https://issues.redhat.com/browse/ACM-9362
Slack thread context https://redhat-internal.slack.com/archives/CUEMEHRA9/p1704996574257619
List all the issues related to this PR
What environments does this code impact?
How was this code tested?
Test steps:
Following the same steps, but deploying MCE stand-alone:
Deploying the stand-alone operator:
assisted-installer
namespace and create quay pull secret in namespaceChecklist
docs
, README, etc)Reviewers Checklist