Skip to content

Conversation

@aevesdocker
Copy link
Contributor

@aevesdocker aevesdocker commented Nov 20, 2024

Description

Light freshness for ECI content and removes references to DD versions that users can no longer download

Note There is a lot of 'allow' violations. Am deliberately choosing to ignore them this time.

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify
Copy link

netlify bot commented Nov 20, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 04fc4d4
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/67488834d233b70008fb796a
😎 Deploy Preview https://deploy-preview-21475--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

It uses a variety of advanced techniques to harden container isolation, but without impacting developer productivity. It is available with [Docker Desktop 4.13.0 and later](/manuals/desktop/release-notes.md).
It uses a variety of advanced techniques to harden container isolation, but without impacting developer productivity.

These techniques include:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have remove this from the top because it is repeated in more detail down below

@aevesdocker aevesdocker marked this pull request as ready for review November 20, 2024 14:10
@aevesdocker aevesdocker requested a review from a team November 20, 2024 14:10
@aevesdocker aevesdocker requested a review from dvdksn November 26, 2024 10:12
Privileged containers `docker run --privileged ...` are insecure because they
give the container full access to the Linux kernel. That is, the container runs
as true root with all capabilities enabled, seccomp and AppArmor restrictions
as true root with all capabilities enabled, Seccomp and AppArmor restrictions
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
as true root with all capabilities enabled, Seccomp and AppArmor restrictions
as true root with all capabilities enabled, seccomp and AppArmor restrictions

@aevesdocker aevesdocker requested a review from dvdksn November 28, 2024 14:51
Artifactory
Autotest
Azure
Berkely
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually this is a typo, shouldn't be in the vocabulary. The correct spelling is Berkeley

@aevesdocker aevesdocker requested a review from dvdksn November 28, 2024 15:11
@aevesdocker aevesdocker merged commit b26161d into docker:main Nov 28, 2024
13 of 14 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.

2 participants