-
Notifications
You must be signed in to change notification settings - Fork 1.2k
nvidia-rootfs: only copy kata-opa
if AGENT_POLICY
is enabled
#11407
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
nvidia-rootfs: only copy kata-opa
if AGENT_POLICY
is enabled
#11407
Conversation
0100d90
to
86feae9
Compare
86feae9
to
8c22c7b
Compare
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.
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:-} |
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.
@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"?
AGENT_POLICY=${AGENT_POLICY:-} | |
AGENT_POLICY="${AGENT_POLICY:-no}" |
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.
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>
8c22c7b
to
d6c4502
Compare
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.
lgtm, thanks @Champ-Goblem!
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.
LGTM. Thanks!
In the nvidia rootfs build, only copy in
kata-opa
ifAGENT_POLICY
is enabled. This fixes builds whenAGENT_POLICY
is disabled and opa is not built.