-
Notifications
You must be signed in to change notification settings - Fork 260
[NPM] Update NPM base image to Distroless #3143
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
Conversation
491fb4c to
44a8801
Compare
44a8801 to
eba581e
Compare
| az aks create --no-ssh-key \ | ||
| --resource-group $(RESOURCE_GROUP) \ | ||
| --name $(AZURE_CLUSTER) \ | ||
| --network-plugin azure |
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.
going to need a \
| --resource-group $(RESOURCE_GROUP) \ | ||
| --name $(AZURE_CLUSTER) \ | ||
| --network-plugin azure | ||
| ----os-sku $(OS_SKU) |
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.
too many - :)
|
@rbtr Do you have any advice for the docker file? |
|
This looks good, it's very similar to what I did for CNS |
| ARG VERSION | ||
| ARG NPM_AI_PATH | ||
| ARG NPM_AI_ID | ||
| RUN apt-get update && apt-get install -y iptables ipset ca-certificates conntrack grep && apt-get autoremove -y && apt-get clean |
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.
make this a separate build stage to keep the concerns distinct and so it can parallelize - let the Go builder only build Go and we'll make an iptables target to get those bits.
move to something like
FROM mariner-core AS iptables
RUN tdnf install -y iptables etcthen change your copies to --from iptables
|
This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days |
|
This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days |
|
This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days |
|
Pull request closed due to inactivity. |
|
This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days |
|
Pull request closed due to inactivity. |
Reason for Change:
Updated NPM base image to Distroless.
Issue Fixed:
Updating npm base image eliminated the segmentation error we would see with ~20k chains added in iptables-nft with base image Ubuntu 20.04
Requirements:
Notes: