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

update helm chart documentation about guac-viz deployment #40

Merged
merged 3 commits into from
Oct 11, 2023

Conversation

pxp928
Copy link
Member

@pxp928 pxp928 commented Oct 11, 2023

fixes #39

Copy link
Contributor

@sunnyyip sunnyyip left a comment

Choose a reason for hiding this comment

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

LGTM

@sunnyyip
Copy link
Contributor

You want to bump the version for the next release.
https://github.com/kusaridev/helm-charts/blob/main/charts/guac/Chart.yaml#L12

Signed-off-by: pxp928 <[email protected]>
@pxp928
Copy link
Member Author

pxp928 commented Oct 11, 2023

version updated

@sunnyyip sunnyyip merged commit f239447 into kusaridev:main Oct 11, 2023
2 checks passed
@pxp928 pxp928 deleted the fix/update-doc-viz branch October 11, 2023 14:33
@@ -37,6 +37,10 @@ The full GUAC component deployment is a set of asynchronous services that combin
- **OSV Certifier**: This certifier gathers OSV vulnerability information from
[osv.dev](https://osv.dev/) about packages.

- **GUAC Visualizer**: The GUAC Visualizer is an experimental utility that can be used to interact with GUAC services. It acts as a way to visualize the software supply chain graph, as well as a means to explore the supply chain and prototype policies.

**Note**: you will need to port-forward to the graphql service for the visualizer to work.
Copy link

Choose a reason for hiding this comment

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

is this still true? I thought this was only the case when running graphql in cluster and visualizer locally

Copy link
Member Author

Choose a reason for hiding this comment

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

oh right, that would not be needed.

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.

Documentation out of date: helm chart installs visualizer
3 participants