-
Notifications
You must be signed in to change notification settings - Fork 241
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
internal/providers: Run platform Init function before fetching config #1155
Conversation
Marked as draft while untested |
b439c5b
to
eea87aa
Compare
This function has been dead since 198184c.
Re second commit message, let's drop |
The value of initialBackoff is increased before the wait is ever performed. This means that we never actually wait for the default initialBackoff time. Move it to after the wait but also increase the current default to mirror what was happening with the previous functionality.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM generally.
Updated & marking ready for review. |
81826a2
to
91f8428
Compare
On AWS we need to run the platform-specific Init function before we fetch the configuration. This Init function is attempting to fetch the S3RegionHint which we need to set to the proper value before attempting to fetch any S3 resources (the region hint defaults to us-east-1 which would cause failures when running EC2 instances in non-standard partitions that are attempting to fetch merge / replacement configurations from S3 resources). Fixes [RHBZ 1892521](https://bugzilla.redhat.com/show_bug.cgi?id=1892521) Fixes [RHBZ 1902996](https://bugzilla.redhat.com/show_bug.cgi?id=1902996)
On AWS we need to run the platform-specific Init function before we
fetch the configuration. This Init function is attempting to fetch the
S3RegionHint which we need to set to the proper value before attempting
to fetch any S3 resources (the region hint defaults to us-east-1 which
would cause failures when running EC2 instances in non-standard
partitions that are attempting to fetch merge / replacement
configurations from S3 resources).
Fixes RHBZ 1892521
Fixes RHBZ 1902996