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

fix(grafana): use relative dashboard path (backport #874) #876

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jun 7, 2024

Welcome to Cryostat! 👋

Before contributing, make sure you have:

  • Read the contributing guidelines
  • Linked a relevant issue which this PR resolves
  • Linked any other relevant issues, PR's, or documentation, if any
  • Resolved all conflicts, if any
  • Rebased your branch PR on top of the latest upstream main branch
  • Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
  • Signed all commits: git commit -S -m "YOUR_COMMIT_MESSAGE"

Related to #726

Description of the change:

Sets the GRAFANA_DASHBOARD_EXT_URL to the simple relative /grafana/ path, rather than the full Route URL.

Motivation for the change:

Since Grafana and Cryostat are now deployed together behind a common auth proxy instance, the client accesses Grafana simply by navigating to /grafana/ on the Cryostat UI URL. This change simplifies the setup so that there are now no more references to the full Route URL in the Deployment, so it is overall more dynamic and flexible. This will support cases where the user wishes to deploy Cryostat on a custom domain or custom subpath.


This is an automatic backport of pull request #874 done by [Mergify](https://mergify.com).

@andrewazores
Copy link
Member

/build_test

@andrewazores
Copy link
Member

Copy link

github-actions bot commented Jun 7, 2024

/build_test : At least one test failed ❌.
View Actions Run.

@andrewazores andrewazores force-pushed the mergify/bp/cryostat-v3.0/pr-874 branch from 35bfaa8 to 9674189 Compare June 7, 2024 19:28
@andrewazores andrewazores requested a review from a team as a code owner June 7, 2024 19:28
@andrewazores
Copy link
Member

/build_test

@andrewazores andrewazores force-pushed the mergify/bp/cryostat-v3.0/pr-874 branch from 9674189 to adf13ce Compare June 7, 2024 19:29
@andrewazores
Copy link
Member

/build_test

@andrewazores
Copy link
Member

Copy link

github-actions bot commented Jun 7, 2024

/build_test completed successfully ✅.
View Actions Run.

@andrewazores andrewazores force-pushed the mergify/bp/cryostat-v3.0/pr-874 branch from adf13ce to 8032351 Compare June 7, 2024 19:45
@andrewazores andrewazores merged commit 1bf4a2c into cryostat-v3.0 Jun 7, 2024
4 of 5 checks passed
@mergify mergify bot deleted the mergify/bp/cryostat-v3.0/pr-874 branch June 7, 2024 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant