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 example policies, UI policy walkthrough, update config #157

Merged
merged 3 commits into from
Oct 14, 2022

Conversation

ltagliaferri
Copy link
Collaborator

@ltagliaferri ltagliaferri commented Oct 14, 2022

Resolves #116 and #153

Also updating config to less frequent crawling cadence.

Signed-off-by: Lisa Tagliaferri

Signed-off-by: Lisa Tagliaferri <lisa.tagliaferri@gmail.com>
@netlify
Copy link

netlify bot commented Oct 14, 2022

Deploy Preview for ornate-narwhal-088216 ready!

Name Link
🔨 Latest commit 26150b2
🔍 Latest deploy log https://app.netlify.com/sites/ornate-narwhal-088216/deploys/6349d612e666d90008ad81b5
😎 Deploy Preview https://deploy-preview-157--ornate-narwhal-088216.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 settings.

Signed-off-by: Lisa Tagliaferri <lisa.tagliaferri@gmail.com>
@@ -55,7 +55,7 @@ rel = "sitemap"
maxAge = "10s"

[sitemap]
changefreq = "hourly"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

made this less frequent as we are no expecting hourly changes right now

@@ -26,16 +26,10 @@ To install `chainctl`, we’ll use the `curl` command to pull the application do
curl -o chainctl "https://dl.enforce.dev/chainctl_$(uname -s)_$(uname -m)"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@katjuell I made a number of updates to the quickstart guide based on changes to the onboarding doc

@@ -243,4 +240,13 @@ chainctl cluster uninstall
kind delete cluster --name enforce-demo
```

To learn more about Chainguard Enforce, please review our documentation and other resources on Chainguard Academy.
## Learn more
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@amdawson — I added a number of links to the UI doc for learning more about Chainguard Enforce. Let me know if there are others we should highlight right now. I think once I know what changes you want to the arch diagram we can add that link too

Copy link
Collaborator

@SharpRake SharpRake left a comment

Choose a reason for hiding this comment

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

Just one minor comment, otherwise I think this looks good

Signed-off-by: Lisa Tagliaferri <lisa.tagliaferri@gmail.com>
Copy link
Collaborator

@SharpRake SharpRake left a comment

Choose a reason for hiding this comment

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

Just the one comment about getting rid of "below". Otherwise all these changes look good to me.

@ltagliaferri ltagliaferri merged commit 7424bbb into main Oct 14, 2022
@ltagliaferri ltagliaferri deleted the policy-ui branch October 15, 2022 13:08
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.

Document more Enforce for k8s policy examples
2 participants