Skip to content

chore: move to GitHub Actions (cherry-pick observatorium/api#915) - #62

Merged
philipgough merged 11 commits into
rhobs:rhobs-obs-api-konfluxfrom
redhat-chai-bot:chai-bot/move-to-github-actions
Oct 1, 2026
Merged

philipgough merged 11 commits into
rhobs:rhobs-obs-api-konfluxfrom
redhat-chai-bot:chai-bot/move-to-github-actions

Conversation

@redhat-chai-bot

Copy link
Copy Markdown

Cherry-pick of upstream observatorium/api#915 to add GitHub Actions CI workflow.

Changes

  • Adds .github/workflows/ci.yaml with 5 CI jobs: build, lint, test, test-e2e, generate
  • Push branch trigger adjusted from main to rhobs-obs-api-konflux to match the fork's default branch

Upstream commit: fbeeddd0d20c

Jira: RHOBS-1792


AI-generated. Review for accuracy.

@philipgough requested via Chai Bot

@redhat-chai-bot
redhat-chai-bot force-pushed the chai-bot/move-to-github-actions branch from a15d264 to 5ab630c Compare September 30, 2026 14:26

@philipgough philipgough left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

/lgtm

@redhat-chai-bot
redhat-chai-bot force-pushed the chai-bot/move-to-github-actions branch from e148421 to 184fc2f Compare September 30, 2026 16:12
andreasgerstmayr and others added 11 commits October 1, 2026 08:14
Signed-off-by: Andreas Gerstmayr <agerstmayr@redhat.com>
Cherry-pick of upstream observatorium/api@c7d00a8

* Update github.com/prometheus/prometheus to 0.313.1
* Update github.com/prometheus-community/prom-label-proxy to 0.14.0
* Update metrics API to use new PromQL parser interface
* Update Go builder images
* Update e2e test
- Remove otelhttp.WithRouteTag() calls (removed in otelhttp v0.69+),
  pass handlers directly to route registrations
- Remove authentication.EnforceAuthentication() call (function was
  deleted in PR stolostron#57 mtls-revert)
- Fix logsv1.NewHandler call: move individual HandlerOption args into
  the logsOpts slice to avoid duplicate arguments
Run make generate to include the --throttle.enable flag documentation
that was added in the throttle feature (PR stolostron#59).
…pstream observatorium#872)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Run golangci-lint --fix to apply gofumpt, gci, and goimports formatters,
fixing the CI lint job that detected uncommitted formatting changes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… upstream observatorium#872)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Bump github.com/efficientgo/e2e from v0.14.1-0.20251204144843-44570aa52a90
to v0.14.1-0.20260204162810-8c75b1e33ef9 to match the upstream rhobs/api
version. The newer version properly initializes the prometheus/common
model.ValidationScheme, fixing a panic in e2e tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
golangci-lint v2.7.0 was built with Go 1.25 and cannot lint code
targeting Go 1.26.0. Upgrade to v2.14.0 which supports Go 1.26.

Also adjust .golangci.yml:
- Remove goconst linter (noisy on test string literals, was never
  enforced since lint was broken)
- Exclude SA1019 for httputil.ReverseProxy.Director (deprecated in
  Go 1.26, migration to Rewrite tracked separately)
@redhat-chai-bot
redhat-chai-bot force-pushed the chai-bot/move-to-github-actions branch from 31135fe to 2210a08 Compare October 1, 2026 08:28
@philipgough
philipgough merged commit 90ebc74 into rhobs:rhobs-obs-api-konflux Oct 1, 2026
4 of 5 checks passed
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.

4 participants