-
Notifications
You must be signed in to change notification settings - Fork 908
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
Release 24.4.1 #5966
Release 24.4.1 #5966
Conversation
The previous way was preventing passing multiple (or no) args.
get_meta_doc() is no longer used, but there may be custom modules that still try to reference it. Don't break them.
I think we still want the 503 fix here, as well as this one. |
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.
.
If the server is busy, no need to fail. Add type hints to adjacent code paths and necessary refactors. Fixes canonicalGH-5577 --------- Co-authored-by: Brett Holman <brett.holman@canonical.com>
There are use cases where we may want to recover or not treat a failure of this function as an error. This change lets the caller decide what to do. Logging was added to callsites that didn't already have it. Fixes canonicalGH-5971
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.
Thanks @TheRealFalcon!
ChangeLog
Outdated
@@ -1,3 +1,10 @@ | |||
24.4.1 | |||
- fix: Don't log errro in wait_for_url (#5972) |
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.
s/errro/error/
…anonical#5976) The function "_should_wait_via_user_data()" wasn't properly handling user data that has a header that starts with #cloud-config, but isn't cloud-config, like #cloud-config-archive Fixes canonicalGH-5975
Bump the version in cloudinit/version.py to 24.4.1 and update ChangeLog.
@holmanb , added one more cherry-pick |
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.
Thanks @TheRealFalcon!
No description provided.