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

[new product] opa gatekeeper #5014

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rawlingsj
Copy link

I manually filled in some previous releases, is that right? If so how far back should we go or are these versions enough?

Copy link

welcome bot commented Apr 20, 2024

Thank you for opening this pull request 👍. If you are not familiar with the project, please check out our Contributing Guidelines and our Guiding Principles. Also take a look at our Hacking Guide if you intend to work on site internals.

Signed-off-by: James Rawlings <jrawlings@chainguard.dev>
a small try to fix regex
Comment on lines +6 to +7
alternate_urls:
- /opa-gatekeeper
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
alternate_urls:
- /opa-gatekeeper

Copy link
Member

@captn3m0 captn3m0 left a comment

Choose a reason for hiding this comment

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

Haven't checked the dates, but a few other suggestions.

@@ -0,0 +1,52 @@
---
title: opa-gatekeeper
category: app
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
category: app
category: server-app

auto:
methods:
- git: https://github.com/open-policy-agent/gatekeeper.git
regex: ^v(?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)(-ce)?$
Copy link
Member

Choose a reason for hiding this comment

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

I didn't see any tags ending with -ce so the default regex should just work for this?

Suggested change
regex: ^v(?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)(-ce)?$


---

> [opa-gatekeeper](https://open-policy-agent.github.io/) Gatekeeper - Policy Controller for Kubernetes
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> [opa-gatekeeper](https://open-policy-agent.github.io/) Gatekeeper - Policy Controller for Kubernetes
> [OPA Gatekeeper](https://open-policy-agent.github.io/gatekeeper/website/) - is a policy controller for Kubernetes that helps enforce policies and strengthen governance.

---

> [opa-gatekeeper](https://open-policy-agent.github.io/) Gatekeeper - Policy Controller for Kubernetes
Supported versions are (current) and n-1 major.minor releases
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Supported versions are (current) and n-1 major.minor releases
The latest two releases are supported with bug and security fixes.

methods:
- git: https://github.com/open-policy-agent/gatekeeper.git
regex: ^v(?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)(-ce)?$

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- identifiers:
- repology: gatekeeper
- purl: pkg:github/open-policy-agent/gatekeeper
- purl: pkg/docker/openpolicyagent/gatekeeper
- purl: pkg/docker/chainguard/gatekeeper
- purl:oci/gatekeeper?repository_url=cgr.dev/chainguard

@captn3m0
Copy link
Member

I manually filled in some previous releases, is that right? If so how far back should we go or are these versions enough?

Whatever is helpful to the user. I think last 5 is good enough for a first version. We can always add later if that feels more helpful.

@captn3m0 captn3m0 self-assigned this May 20, 2024
@captn3m0 captn3m0 added the new-product This PR adds a new product to the website. label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-product This PR adds a new product to the website.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants