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

[ENHANCEMENT] Update dependencies and improve docs #21

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

ibakshay
Copy link
Contributor

@ibakshay ibakshay commented Oct 21, 2024

Changes

  • Upgraded to Go 1.23
  • Updated all project dependencies
  • Enhanced README
  • Refreshed config/samples manifests
  • Updated CRDs in config/crd/bases to reflect new schema
  • Added optional perses-server-url flag for localhost testing:
    flag.StringVar(&persesServerURL, "perses-server-url", "", "The Perses backend server URL")

fixes #16 cloudoperators/greenhouse-extensions#419

@ibakshay ibakshay force-pushed the akshay/update-dependencies branch 2 times, most recently from 254f17c to 249f796 Compare October 21, 2024 11:39
Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
@ibakshay ibakshay changed the title Update dependencies and improve docs [ENHANCEMENT] Update dependencies and improve docs Oct 21, 2024
@jgbernalp
Copy link
Contributor

@ibakshay thank you for your contribution!. Can you please take a look at the tests, they might need an update.

Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
@ibakshay
Copy link
Contributor Author

ibakshay commented Oct 22, 2024

@ibakshay thank you for your contribution!. Can you please take a look at the tests, they might need an update.

My pleasure. Thanks a lot for your review. The test is failing due to this serialization issue. I have already created a PR in prometheus/common repository for the fix. The test should be successfully after PR in prometheus/common gets merged.

Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
Signed-off-by: Akshay Iyyadurai Balasundaram <[email protected]>
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.

Needs Dependency Updates
2 participants