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

tools: update go and add goyacc and drone-cli #3979

Closed
wants to merge 2 commits into from

Conversation

electron0zero
Copy link
Member

What this PR does:

update tools image to include goyaac and drone-cli,
also update go to 1.23 in the tools image, required to update go version in tempo.

once we have new image with these tools, we can stop using loki-build-image and move these commands to use our own build image.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

build/tools.mk Outdated
@@ -12,7 +12,7 @@ TOOL_DIR ?= tools
TOOL_CONFIG ?= $(TOOL_DIR)/tools.go

TOOLS_IMAGE ?= grafana/tempo-ci-tools
TOOLS_IMAGE_TAG ?= main-5e4eafd
TOOLS_IMAGE_TAG ?= latest
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
TOOLS_IMAGE_TAG ?= latest
TOOLS_IMAGE_TAG ?= main-5e4eafd

Copy link
Contributor

This PR has been automatically marked as stale because it has not had any activity in the past 60 days.
The next time this stale check runs, the stale label will be removed if there is new activity. This pull request will be closed in 15 days if there is no new activity.
Please apply keepalive label to exempt this Pull Request.

@github-actions github-actions bot added the stale Used for stale issues / PRs label Nov 27, 2024
@electron0zero electron0zero added keepalive Label to exempt Issues / PRs from stale workflow and removed stale Used for stale issues / PRs labels Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keepalive Label to exempt Issues / PRs from stale workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant