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

feat(ci): Add CI-check for pop-deploys #2184

Merged
merged 3 commits into from
Jun 6, 2023
Merged

feat(ci): Add CI-check for pop-deploys #2184

merged 3 commits into from
Jun 6, 2023

Conversation

TBS1996
Copy link
Contributor

@TBS1996 TBS1996 commented Jun 5, 2023

This should prevent deploying pop-relays before it has passed all the CI tests.

#skip-changelog

@TBS1996 TBS1996 requested a review from a team June 5, 2023 12:24
relay/src/cli.rs Outdated Show resolved Hide resolved
# Required for checkruns.
GITHUB_TOKEN: "{{SECRET:[devinfra-github][token]}}"
timeout: 1800
elastic_profile_id: relay
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be the same profile as processing relays?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's now relay-pop to align with the other pop jobs

Copy link
Contributor Author

@TBS1996 TBS1996 Jun 5, 2023

Choose a reason for hiding this comment

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

sorry for the noise! I should have made it draft

@TBS1996 TBS1996 marked this pull request as draft June 5, 2023 12:47
@TBS1996 TBS1996 marked this pull request as ready for review June 5, 2023 15:49
Copy link
Member

@jjbayer jjbayer left a comment

Choose a reason for hiding this comment

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

As discussed offline, another approach would be to make sure that PoPs can only deploy versions that have already been deployed to processing relays. But this should work too!

@TBS1996 TBS1996 merged commit a63cc43 into master Jun 6, 2023
@TBS1996 TBS1996 deleted the pop-ci-check branch June 6, 2023 14:00
jan-auer added a commit that referenced this pull request Jun 14, 2023
* master: (32 commits)
  feat(txname): Mark all URL transactions as sanitized (#2210)
  fix(profiles): Enforce profile quotas on metrics buckets (#2212)
  revert(spans): Move span metrics back to transactions namespace (#2206)
  instr(metrics): Add values to invalid metric errors (#2201)
  ref(spans): Move span metrics to spans namespace (#2205)
  chore(protocol): Add missing py/changelog entry for lock property (#2203)
  build(py): Move pytest.ini to tests directory (#2204)
  ref: Use serde collect_str to avoid allocations (#2149)
  chore(gocd): Auto approve deploy relay-pop on check success (#2200)
  test(document-pii): Fix flaky test (#2198)
  feat(spans): Extract transaction `user` into span's databag (#2197)
  feat(spans): Extract `release` tag from the transaction to the span (#2189)
  ref(metrics-extraction): Split span and transaction metric extraction (#2191)
  ref(crons): Support GET in the check-in endpoint (#2192)
  ref(spans): Introduce opinionated casing in span metrics (#2193)
  chore(py): Bump `requests` lib version (#2194)
  doc: document pii fields (#1972)
  feat: Added cron specific endpoint (#2153)
  feat(profiling): Extract app identifier from app context (#2172)
  feat(ci): Add CI-check for pop-deploys  (#2184)
  ...
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.

3 participants