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

Polling the presence of gem to trigger gitlab job #3161

Merged
merged 4 commits into from
Sep 30, 2023

Conversation

TonyCTHsu
Copy link
Contributor

Motivation:

Building lib injection package depends on the gem ddtrace being published at rubygems.org.

During ddtrace release, the process is triggered by a tag push event from the repository but failed since the gem has not yet been published. It needs to be manually retriggered after the gem is published to rubygems.org successfully.

What does this PR do?

Create a job that makes polling request to rubygems.org and blocks to pipeline until the the gem has been published.

@TonyCTHsu TonyCTHsu requested a review from a team as a code owner September 27, 2023 10:24
@TonyCTHsu TonyCTHsu added the dev/ci Involves CircleCI, GitHub Actions, or GitLab label Sep 27, 2023
Copy link
Contributor

@randomanderson randomanderson left a comment

Choose a reason for hiding this comment

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

👍

@github-actions github-actions bot removed the dev/ci Involves CircleCI, GitHub Actions, or GitLab label Sep 29, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #3161 (95e0066) into master (1ef3cf5) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3161      +/-   ##
==========================================
- Coverage   98.14%   98.14%   -0.01%     
==========================================
  Files        1247     1247              
  Lines       71703    71703              
  Branches     3338     3338              
==========================================
- Hits        70371    70370       -1     
- Misses       1332     1333       +1     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@marcotc marcotc left a comment

Choose a reason for hiding this comment

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

I think, ideally, this step should be triggered after rubygems.org receives the new gem version.
rubygems.org publishing is not automatic today, so it's not possible to trigger this PR's job after the rubygems.org. But when we automate rubygems.org publishing, we should trigger this PR's job automatically, no through polling.

@TonyCTHsu TonyCTHsu merged commit 62e6c52 into master Sep 30, 2023
176 checks passed
@TonyCTHsu TonyCTHsu deleted the tonycthsu/automate-lib-injection-package-build branch September 30, 2023 06:49
@github-actions github-actions bot added this to the 1.15.0 milestone Sep 30, 2023
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