Skip to content

Conversation

Champ-Goblem
Copy link
Contributor

In the nvidia rootfs build, only copy in kata-opa if AGENT_POLICY is enabled. This fixes builds when AGENT_POLICY is disabled and opa is not built.

@Champ-Goblem Champ-Goblem force-pushed the fix/nvidia-rootfs-only-copy-opa-when-agent-policy-enabled branch from 0100d90 to 86feae9 Compare June 10, 2025 18:41
@sprt sprt requested review from danmihai1 and Redent0r June 10, 2025 18:42
@Champ-Goblem Champ-Goblem force-pushed the fix/nvidia-rootfs-only-copy-opa-when-agent-policy-enabled branch from 86feae9 to 8c22c7b Compare June 10, 2025 18:48
Copy link
Contributor

@Apokleos Apokleos left a comment

Choose a reason for hiding this comment

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

Thx @Champ-Goblem LGTM

@@ -15,6 +15,7 @@ readonly SCRIPT_DIR="${script_dir}/nvidia"
# This will control how much output the inird/image will produce
DEBUG=""
KBUILD_SIGN_PIN=${KBUILD_SIGN_PIN:-}
AGENT_POLICY=${AGENT_POLICY:-}
Copy link
Member

Choose a reason for hiding this comment

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

@Champ-Goblem, just for the sake of consistency with what we have as part of the other rootfs scripts ...
Could you sete the default policy here to "no"?

Suggested change
AGENT_POLICY=${AGENT_POLICY:-}
AGENT_POLICY="${AGENT_POLICY:-no}"

Copy link
Member

Choose a reason for hiding this comment

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

I've fixed this and force-pushed to your branch, @Champ-Goblem!

In the nvidia rootfs build, only copy in `kata-opa` if `AGENT_POLICY` is enabled. This fixes
builds when `AGENT_POLICY` is disabled and opa is not built.

Signed-off-by: Champ-Goblem <cameron@northflank.com>
@fidencio fidencio force-pushed the fix/nvidia-rootfs-only-copy-opa-when-agent-policy-enabled branch from 8c22c7b to d6c4502 Compare June 11, 2025 09:25
Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @Champ-Goblem!

Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@stevenhorsman stevenhorsman merged commit c8fcda0 into kata-containers:main Jun 11, 2025
498 of 545 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants