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

feat: add grafana rock #39

Merged
merged 5 commits into from
Aug 23, 2023
Merged

feat: add grafana rock #39

merged 5 commits into from
Aug 23, 2023

Conversation

lucabello
Copy link
Collaborator

@lucabello lucabello commented Aug 16, 2023

Ping the @canonical/rocks team.


Description

This PR adds the Grafana ROCK to the oci-factory.

Related issues

#43

@lucabello lucabello marked this pull request as ready for review August 16, 2023 14:01
@cjdcordeiro cjdcordeiro mentioned this pull request Aug 17, 2023
13 tasks
Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

Thanks for this @lucabello !

Given the comment from #43 (comment), can you please expand the docs to highlight the changes being introduced with this release? I.e. Docker image vs ROCK

@cjdcordeiro cjdcordeiro self-assigned this Aug 18, 2023
@cjdcordeiro cjdcordeiro merged commit c337c5f into canonical:main Aug 23, 2023
14 checks passed
@cjdcordeiro
Copy link
Collaborator

@lucabello the image didn't pass the vulnerability tests: https://github.com/canonical/oci-factory/actions/runs/5947996718

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.

3 participants