Skip to content

[terraform] Update retry logic to handle DNS-related errors more gracefully - #713

Merged
Samruddhi Khandale (samruddhikhandale) merged 3 commits into
devcontainers:mainfrom
alexander-smolyakov:terraform-update_retry_logic
Oct 4, 2023
Merged

[terraform] Update retry logic to handle DNS-related errors more gracefully#713
Samruddhi Khandale (samruddhikhandale) merged 3 commits into
devcontainers:mainfrom
alexander-smolyakov:terraform-update_retry_logic

Conversation

@alexander-smolyakov

Copy link
Copy Markdown
Contributor

Feature name:

  • ghcr.io/devcontainers/features/terraform

Description:

This PR aims to update the retry logic to handle the gpg: keyserver receive failed: Server indicated a failure error more gracefully.

The gpg returns the keyserver receive failed: Server indicated a failure error if something goes wrong with DNS-related stuff. It looks like gpg could use several libraries for working with DNS servers based on macros values. Related code:

This means that gpg could work with DNS servers differently than other tools.

The fix is to try to get the IP address of the GPG keyserver and explicitly pass it to the gpg tool to avoid DNS-related issues.

Changelog:

  • Retry logic updated: Additional tries with explicitly passing the keyserver IP address were added;
  • Thednsutils package added to dependencies;

Attached related issue:

Checklist:

  • Checked that applied changes work as expected

- Add `dnsutils` package to dependencies
- Update retry logic to handle DNS-related errors more gracefully

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thank you! 🎉

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.

2 participants