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.

Sign up for GitHub

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

Jump to bottom

Add retries to choco install steps #5779

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Add retries to choco install steps #5779

merged 1 commit into from
Oct 15, 2024

Conversation

rowlandgeoff
Copy link
Contributor

@rowlandgeoff rowlandgeoff commented Oct 14, 2024 •
edited
Loading

Add retries to a few more Windows setup steps in the agent CI to try to reduce flakiness.

Rel: https://github.com/elastic/ingest-dev/issues/4245

@rowlandgeoff rowlandgeoff added skip-changelog backport-8.15 Automated backport to the 8.15 branch with mergify backport-8.x Automated backport to the 8.x branch with mergify labels Oct 14, 2024
@rowlandgeoff rowlandgeoff self-assigned this Oct 14, 2024
@elastic-sonarqube Elastic SonarQube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

blakerouse
blakerouse approved these changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@blakerouse blakerouse left a comment

Choose a reason for hiding this comment

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

Looks perfect. Merge it!

@rowlandgeoff rowlandgeoff marked this pull request as ready for review October 15, 2024 14:55
@rowlandgeoff rowlandgeoff requested a review from a team as a code owner October 15, 2024 14:55
@rowlandgeoff rowlandgeoff merged commit e174d83 into main Oct 15, 2024
17 checks passed
@rowlandgeoff rowlandgeoff deleted the windowsInstallRetry branch October 15, 2024 14:55
mergify bot pushed a commit that referenced this pull request Oct 15, 2024
@rowlandgeoff @mergify
Add retries to choco install steps (#5779)
ccdc719 
(cherry picked from commit e174d83)
mergify bot pushed a commit that referenced this pull request Oct 15, 2024
@rowlandgeoff @mergify
Add retries to choco install steps (#5779)
c996e1d 
(cherry picked from commit e174d83)
This was referenced Oct 15, 2024
Merged
Merged
pierrehilbert pushed a commit that referenced this pull request Oct 15, 2024
@mergify @rowlandgeoff
Add retries to choco install steps (#5779) (#5793)
107b90e 
(cherry picked from commit e174d83)

Co-authored-by: Geoff Rowland <70981735+rowlandgeoff@users.noreply.github.com>
rowlandgeoff added a commit that referenced this pull request Oct 18, 2024
@mergify @rowlandgeoff
Add retries to choco install steps (#5779) (#5792)
51ae13b 
(cherry picked from commit e174d83)

Co-authored-by: Geoff Rowland <70981735+rowlandgeoff@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blakerouse blakerouse blakerouse approved these changes

@andrzej-stencel andrzej-stencel Awaiting requested review from andrzej-stencel andrzej-stencel is a code owner automatically assigned from elastic/elastic-agent-control-plane

@michel-laterman michel-laterman Awaiting requested review from michel-laterman michel-laterman is a code owner automatically assigned from elastic/elastic-agent-control-plane

Assignees

@rowlandgeoff rowlandgeoff

Labels
backport-8.x Automated backport to the 8.x branch with mergify backport-8.15 Automated backport to the 8.15 branch with mergify skip-changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rowlandgeoff @blakerouse