Skip to content
This repository has been archived by the owner on Sep 22, 2024. It is now read-only.

Updated jumpbox image to correct repository and removed deprecated arguement #134

Merged
merged 2 commits into from
Nov 13, 2023
Merged

Updated jumpbox image to correct repository and removed deprecated arguement #134

merged 2 commits into from
Nov 13, 2023

Conversation

TBunkley
Copy link
Contributor

@TBunkley TBunkley commented Nov 10, 2023 •
edited
Loading

Type of PR

  • Documentation changes
  • Code changes
  • GitHub Template changes

Purpose of PR

Validation

  • Update documentation or issue referenced above

Issues Closed or Referenced

  • Closes #issue_number (this will automatically close the issue when the PR closes)
  • References #issue_number (this references the issue but does not close with PR)

aflinchb
aflinchb approved these changes Nov 10, 2023
View reviewed changes
Copy link
Contributor

@aflinchb aflinchb left a comment

Choose a reason for hiding this comment

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

Approved w/ a comment to also remove the deprecated prometheus flag from the deploy/webv-local.yaml

deploy/webv/webv.yaml Show resolved Hide resolved
@TBunkley TBunkley merged commit 1ba8d96 into cse-labs:main Nov 13, 2023
2 checks passed
@TBunkley TBunkley deleted the fix-startup-script-and-configuration branch November 13, 2023 16:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@aflinchb aflinchb aflinchb approved these changes

Assignees

@TBunkley TBunkley

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TBunkley @aflinchb