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

Add GHA for building promql-to-scrape #87

Merged
merged 1 commit into from
May 8, 2024

Conversation

taonic
Copy link
Collaborator

@taonic taonic commented May 7, 2024

What was changed

Adding GHA workflow to build https://github.com/temporalio/samples-server/tree/main/cloud/observability/promql-to-scrape and publish to https://github.com/orgs/temporalio/packages

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@taonic taonic force-pushed the taonic/add-GHA-to-build-promql-to-scrape branch 6 times, most recently from ec60964 to 83ff47c Compare May 7, 2024 10:37
@taonic
Copy link
Collaborator Author

taonic commented May 7, 2024

Build was fine but failed when pushing to ghcr since secrets.GITHUB_TOKEN is read-only in pull_request events.
https://github.com/temporalio/samples-server/actions/runs/8983905437/job/24674797050

It shouldn't be the case once it's on main.

@@ -1,11 +1,24 @@
FROM golang:1.21-alpine
FROM --platform=${BUILDPLATFORM:-linux/amd64} golang:1.20 as builder
Copy link
Collaborator

Choose a reason for hiding this comment

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

1.21? probably doesn't matter but just to stay consistent

@taonic taonic force-pushed the taonic/add-GHA-to-build-promql-to-scrape branch from 83ff47c to 22b5bc0 Compare May 7, 2024 22:57
@taonic taonic merged commit cf187d7 into main May 8, 2024
2 of 3 checks passed
@taonic taonic deleted the taonic/add-GHA-to-build-promql-to-scrape branch May 8, 2024 00:02
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