Skip to content
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

Add handling of ENTRYPOINT and CMD when "command" is not in policy #1304

Merged
merged 1 commit into from
May 23, 2022

Conversation

anmaxvl
Copy link
Contributor

@anmaxvl anmaxvl commented Feb 23, 2022

Container images may contain ENTRYPOINT and CMD directives and in
the case when "command" is missing in policy config, that information
needs to be inferred from the image itself.

Signed-off-by: Maksim An maksiman@microsoft.com

@anmaxvl anmaxvl requested a review from a team as a code owner February 23, 2022 00:28
@anmaxvl
Copy link
Contributor Author

anmaxvl commented Feb 23, 2022

@SeanTAllen @KenGordon FYI

@SeanTAllen
Copy link
Contributor

Looks good to me

@anmaxvl anmaxvl force-pushed the policy-image-entrypoint branch from de66cd1 to a4dd146 Compare March 16, 2022 06:47
Container images may contain ENTRYPOINT and CMD directives and in
the case when "command" is missing in policy config, that information
needs to be inferred from the image itself.

Signed-off-by: Maksim An <maksiman@microsoft.com>
@anmaxvl anmaxvl force-pushed the policy-image-entrypoint branch from a4dd146 to 7f3af60 Compare March 16, 2022 06:50
@anmaxvl anmaxvl merged commit ffc5937 into microsoft:master May 23, 2022
@anmaxvl anmaxvl deleted the policy-image-entrypoint branch May 23, 2022 20:19
anmaxvl added a commit that referenced this pull request Feb 7, 2023
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
…osoft#1304)

Container images may contain ENTRYPOINT and CMD directives and in
the case when "command" is missing in policy config, that information
needs to be inferred from the image itself.

Signed-off-by: Maksim An <maksiman@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants