Skip to content

add token-vendor resources to helm chart values#672

Merged
koonpeng merged 3 commits into
mainfrom
koonpeng/tv-resources
May 11, 2026
Merged

add token-vendor resources to helm chart values#672
koonpeng merged 3 commits into
mainfrom
koonpeng/tv-resources

Conversation

@koonpeng
Copy link
Copy Markdown
Contributor

cherry pick from #665.

koonpeng added 2 commits May 11, 2026 16:13
Signed-off-by: Teo Koon Peng <koonpeng@google.com>
Signed-off-by: Teo Koon Peng <koonpeng@google.com>
Comment thread MODULE.bazel.lock
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you need to undo this? Reducing the lockFileVersion seems like a mistake.

Copy link
Copy Markdown
Contributor Author

@koonpeng koonpeng May 11, 2026

Choose a reason for hiding this comment

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

Tried reverting it but doing a bazel build change it back again. Tried checking out main, make sure there are no changes, doing a bazel build still change the lock file version 🤔.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think #667 is made with a newer version of bazel that does not respect .bazelversion.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hmm, I'm worried about merging the change given https://github.com/googlecloudrobotics/core/pull/667/changes suggests we might break the build elsewhere. Maybe you can just split it out of this commit locally and ignore the fact that Bazel is trying to undo it until @ensonic can let us know if any changes are necessary in the workspace for #667?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Signed-off-by: Teo Koon Peng <koonpeng@google.com>
@koonpeng koonpeng enabled auto-merge (squash) May 11, 2026 08:38
@koonpeng koonpeng merged commit 7b9975d into main May 11, 2026
7 checks passed
@koonpeng koonpeng deleted the koonpeng/tv-resources branch May 11, 2026 08:43
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.

2 participants