Skip to content

Conversation

@rwsu
Copy link
Contributor

@rwsu rwsu commented Jun 6, 2023

Introduces AGENT_USE_APPLIANCE_MODEL environment variable. Setting this variable to true will generate an agent ISO containing the unconfigured ignition.

Depends on: openshift/installer#7186

Introduces AGENT_USE_APPLIANCE_MODEL environment variable.
Setting this variable to true will generate an agent ISO containing
the unconfigured ignition.
@openshift-ci openshift-ci bot requested review from bfournie and derekhiggins June 6, 2023 03:06
@openshift-ci
Copy link

openshift-ci bot commented Jun 6, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign derekhiggins for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

function create_factory_image() {
"${openshift_install}" --dir="${asset_dir}" --log-level=debug agent create cluster-manifests
"${openshift_install}" --dir="${asset_dir}" --log-level=debug agent create unconfigured-ignition
oc adm release info --image-for=machine-os-images --insecure=true $OPENSHIFT_RELEASE_IMAGE
Copy link
Member

Choose a reason for hiding this comment

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

This looks like a typo?

"${openshift_install}" --dir="${asset_dir}" --log-level=debug agent create cluster-manifests
"${openshift_install}" --dir="${asset_dir}" --log-level=debug agent create unconfigured-ignition
oc adm release info --image-for=machine-os-images --insecure=true $OPENSHIFT_RELEASE_IMAGE
base_iso_url=$(oc adm release info --image-for=machine-os-images --insecure=true $OPENSHIFT_RELEASE_IMAGE)
Copy link
Member

Choose a reason for hiding this comment

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

nit: more than a base_iso_url it's a pullspec

oc image extract --path /coreos/coreos-x86_64.iso:$HOME/.cache/agent/image_cache --confirm $base_iso_url
local agent_iso_abs_path="$(realpath "${OCP_DIR}")"
podman run --privileged --rm -v /run/udev:/run/udev -v "${agent_iso_abs_path}:${agent_iso_abs_path}" -v "$HOME/.cache/agent/image_cache/:$HOME/.cache/agent/image_cache/" quay.io/coreos/coreos-installer:release iso ignition embed -f -i "${agent_iso_abs_path}/unconfigured-agent.ign" -o "${agent_iso_abs_path}/agent.iso" $HOME/.cache/agent/image_cache/coreos-x86_64.iso
}
Copy link
Member

Choose a reason for hiding this comment

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

Not sure I'm missing something, but how are we going to test/exercise this new mode? I suppose that maybe another set of changes/PR will follow up, to implement the full workflow?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I should have marked this wip. Bob has taken this PR and added the entire workflow here #1533. Will close this one.

@rwsu rwsu changed the title AGENT-611 Add support for unconfigured ignition [wip] AGENT-611 Add support for unconfigured ignition Jun 24, 2023
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 24, 2023
@rwsu
Copy link
Contributor Author

rwsu commented Jun 24, 2023

Closing this PR. It has been incorporated into #1533

@rwsu rwsu closed this Jun 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants