-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Don't default adding MIMEBOUNDARY headers when a mixed instances policy is set #7370
Don't default adding MIMEBOUNDARY headers when a mixed instances policy is set #7370
Conversation
6452145
to
2756d58
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: KashifSaadat The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
/lgtm |
@KashifSaadat Thanks a lot for this! I've been waiting for it since 1.12 alpha, with Kops frozen because I use CoreOS and MixedInstancePolicy. I opened this issue but haven't received any return since then. #6863 |
@edsonmarquezani no worries at all, sorry it took so long to fix! |
Thanks for fixing @KashifSaadat Should we cherry pick all the way back to 1.12? |
I sent #7418 to cherry-pick to 1.14, as we want to cut a 1.14 beta soon! |
Awesome thanks! I'll review and LG :) |
@justinsb A cherry-pick from 1.12.x ahead would be welcome, since I don't need to jump right to 1.14, skipping 1.13. Not sure if this is completely safe. |
Ah missed your previous message, yes it may be good to cherry-pick back for those using the older kops releases 👍 |
Cherry pick of #7370 onto release-1.14
Currently the MIMEBOUNDARY header is added if one of the following cases is met:
AdditionalUserData
is set on an Instance GroupMixedInstancesPolicy
is set on an Instance GroupThe latter being set on an IG for a CoreOS nodepool now breaks on provisioning the nodes, as CoreOS doesn't support MIME messages. It is still set as required when
AdditionalUserData
has been provided, but will need follow-up with further investigation to get additional user data scripts working when using CoreOS.