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

(IAC-1597) Increasing MAX_RETRY_COUNT #987

Merged
merged 1 commit into from
May 11, 2021

Conversation

pmcmaw
Copy link

@pmcmaw pmcmaw commented May 6, 2021

Currently CI runs are seeing transient failures in testing on Ubuntu18 as the keyserver is not installed yet. This issue is consistent and this week has been seen on 2/5 nightlies. Bumping the MAx_RETRIES for Ubuntu18 to 10 to see if this resolves the issue.

This issue will be resolved when there are 5 nightly runs consecutively passing without the error being thrown.

Extended change to all OS as the same issue was seen on Debian 9.

@pmcmaw pmcmaw requested a review from a team as a code owner May 6, 2021 16:41
@pmcmaw pmcmaw added the bugfix label May 6, 2021
@pmcmaw
Copy link
Author

pmcmaw commented May 10, 2021

Unsure whether using a conditional to bump RETRIES on Ubuntu18 or bumping retries in general would be the best move here to avoid conditionals and complexity. Feedback from the reviewer would be much appreciated on this.

@Disha-maker
Copy link

Adding information on the errors seen on debian-9 for further investigation.

For the debian-9 under acceptance tests - this is the first error 
Error: /Stage[main]/Apt/Apt::Source[puppetlabs]/Apt::Key[Add key: 6F6B15509CF8E59E6E469F327F438280EF8D349F from Apt::Source puppetlabs]/Apt_key[Add key: 6F6B15509CF8E59E6E469F327F438280EF8D349F from Apt::Source puppetlabs]/ensure: change from 'absent' to 'present' failed: 


Execution of '/usr/bin/apt-key adv --no-tty --keyserver pool.sks-keyservers.net --recv-keys 6F6B15509CF8E59E6E469F327F438280EF8D349F' returned 2: Warning: apt-key output should not be parsed (stdout is not a terminal)

@pmcmaw
Copy link
Author

pmcmaw commented May 11, 2021

The same issue has been seen over the weekend on Debian9 therefore making the decision to bump RETRIES for all OS.

@pmcmaw pmcmaw changed the title (IAC-1597) Increasing MAX_RETRIES for Ubuntu18 (IAC-1597) Increasing MAX_RETRIES May 11, 2021
@pmcmaw pmcmaw changed the title (IAC-1597) Increasing MAX_RETRIES (IAC-1597) Increasing MAX_RETRY_COUNT May 11, 2021
@carabasdaniel carabasdaniel merged commit 176542d into puppetlabs:main May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants