Skip to content

Update naming in internal security policy tool#1166

Merged
anmaxvl merged 1 commit intomicrosoft:masterfrom
SeanTAllen:image-to-container
Sep 20, 2021
Merged

Update naming in internal security policy tool#1166
anmaxvl merged 1 commit intomicrosoft:masterfrom
SeanTAllen:image-to-container

Conversation

@SeanTAllen
Copy link
Contributor

Maksim pointed out that when we added information beyond the image of an image
that the "image" entries in a TOML policy generation file weren't describing
images; the describe containers.

The addition of command line, environment variables, and what not to allow
is a description of a container that should be allowed to be created. The
only image specific bit is the name.

Signed-off-by: Sean T. Allen seanallen@microsoft.com

@SeanTAllen SeanTAllen requested a review from a team as a code owner September 20, 2021 16:40
Maksim pointed out that when we added information beyond the image of an image
that the "image" entries in a TOML policy generation file weren't describing
images; the describe containers.

The addition of command line, environment variables, and what not to allow
is a description of a container that should be allowed to be created. The
only image specific bit is the name.

Signed-off-by: Sean T. Allen <seanallen@microsoft.com>
@anmaxvl anmaxvl merged commit 8f21c11 into microsoft:master Sep 20, 2021
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
Maksim pointed out that when we added information beyond the image of an image
that the "image" entries in a TOML policy generation file weren't describing
images; the describe containers.

The addition of command line, environment variables, and what not to allow
is a description of a container that should be allowed to be created. The
only image specific bit is the name.

Signed-off-by: Sean T. Allen <seanallen@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.

2 participants