-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Test custom machine-os-content image in machine-config-operator@mcbs #24225
Test custom machine-os-content image in machine-config-operator@mcbs #24225
Conversation
I think I understand the hack here. Clever! But let's target this against the mcbs branch file where we're focusing this effort. For now we are not trying to land code in |
ed9875e
to
610adcd
Compare
@petr-muller: 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. |
610adcd
to
344333a
Compare
...perator/config/openshift/machine-config-operator/openshift-machine-config-operator-mcbs.yaml
Show resolved
Hide resolved
1. Import the custom image 2. "build" the machine-os-content from the custom image 3. Such "built" image gets picked up by the `include_built_images: true`-configured `latest` release that gets installed for testing 4. Make sure that `machine-os-content` will not be promoted (presubmits should forbid that but to be safe).
344333a
to
c96ce97
Compare
/retitle Test custom machine-os-content image in machine-config-operator@mcbs |
For clarity, this replaces #24222 |
Hmm, I think the override isn't working:
Versus:
|
It looks like you are inspecting the |
Ahh, got it. OK indeed, the target image looks right:
However, as a side effect of this hack it has a different digested /approve And thanks for doing this! |
^^^ from the config. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cgwalters, petr-muller 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 |
I think there is even blacker magic in ci-operator that would help with that ( |
@petr-muller: Updated the
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. |
This is *mainly* to validate that openshift/release#24225 worked. But, this code may be useful as a sanity check going forward. See also coreos/rpm-ostree#3251 (I also may try to expose e.g. `ex-container` as a feature flag that we can query instead of version-parsing)
This is *mainly* to validate that openshift/release#24225 worked. But, this code may be useful as a sanity check going forward. See also coreos/rpm-ostree#3251 (I also may try to expose e.g. `ex-container` as a feature flag that we can query instead of version-parsing)
/cc @cgwalters