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 AppCat VSHN PostgreSQL and VSHN Redis query to new pricing model #138

Merged
merged 5 commits into from
May 10, 2023

Conversation

wejdross
Copy link
Contributor

@wejdross wejdross commented May 9, 2023

Summary

With the updated billing model we use different naming for the AppCat queries and also match on a different namespace label. The label switch is done so that both the old and new query continue to work. See: vshn/component-appcat#144

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Update tests.
  • Link this PR to related issues.

@wejdross wejdross requested a review from a team as a code owner May 9, 2023 08:22
@wejdross wejdross requested a review from glrf May 9, 2023 08:22
@glrf glrf changed the title updating appcat services to use new query Update AppCat VSHN PostgreSQL and VSHN Redis query to new naming May 9, 2023
@glrf glrf added the change label May 9, 2023
pkg/db/seeds.go Outdated Show resolved Hide resolved
@wejdross wejdross requested a review from glrf May 9, 2023 14:07
pkg/db/seeds/appcat_vshn.promql.tmpl Outdated Show resolved Hide resolved
@wejdross wejdross requested a review from glrf May 10, 2023 07:20
@glrf glrf changed the title Update AppCat VSHN PostgreSQL and VSHN Redis query to new naming Update AppCat VSHN PostgreSQL and VSHN Redis query to new pricing model May 10, 2023
@wejdross wejdross merged commit 735a7f3 into master May 10, 2023
3 checks passed
@wejdross wejdross deleted the updating_appcat_query branch May 10, 2023 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants