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

remove MixedInstancesPolicy from nodeup MIMEBOUNDARY #7091

Closed
Deshke opened this issue Jun 3, 2019 · 0 comments · Fixed by #7370
Closed

remove MixedInstancesPolicy from nodeup MIMEBOUNDARY #7091

Deshke opened this issue Jun 3, 2019 · 0 comments · Fixed by #7370

Comments

@Deshke
Copy link

Deshke commented Jun 3, 2019

1. Describe IN DETAIL the feature/behavior/change you would like to see.

with the merge from #6732, the nodeup script

if len(ig.Spec.AdditionalUserData) > 0 || ig.Spec.MixedInstancesPolicy != nil {
changed to include the MixedInstancesPolicy, but MixedInstancesPolicy does not change anything on the user-data and multipart content breaks coreOS-cloudinit coreos/bugs#2588

Content-Type: multipart/mixed; should be used if there are multiple scripts but there is only one being provided and ig.Spec.MixedInstancesPolicy does not change that.

2. Feel free to provide a design supporting your feature request.

  • Content-Type: multipart/mixed; should only be set if multiple scripts are provided
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 a pull request may close this issue.

1 participant