Skip to content

Conversation

@wrn14897
Copy link
Collaborator

@wrn14897 wrn14897 commented Jun 5, 2025

Ref: HDX-1839

@changeset-bot
Copy link

changeset-bot bot commented Jun 5, 2025

🦋 Changeset detected

Latest commit: 93dc58e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
helm-charts Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

type: application
version: 0.5.2
appVersion: 1.0.0
appVersion: 2.0.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

going forward, we only need to bump the appVersion here (ideally triggered by the upstream)

@wrn14897 wrn14897 changed the title feat: refactor image value + bump default tag to 2.0.0 feat: refactor image value + bump default tag to 2.0.1 Jun 9, 2025
image: "hyperdx/hyperdx-otel-collector:2-beta"
image:
repository: docker.hyperdx.io/hyperdx/hyperdx-otel-collector
tag:
Copy link
Contributor

Choose a reason for hiding this comment

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

We probably want to assign a default tag, yeah? latest?

Copy link
Collaborator Author

@wrn14897 wrn14897 Jun 10, 2025

Choose a reason for hiding this comment

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

the default is appVersion. pinning the version for now since we want to make sure all int + e2e tests pass

teeohhem
teeohhem previously approved these changes Jun 11, 2025
@teeohhem teeohhem self-requested a review June 11, 2025 14:27
@kodiakhq kodiakhq bot merged commit 1541c5f into main Jun 11, 2025
3 of 4 checks passed
@kodiakhq kodiakhq bot deleted the warren/bump-images-tag branch June 11, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants