Skip to content

Comments

Detect multipart mime and ignore it#823

Closed
gabriel-samfira wants to merge 1 commit intoflatcar:mainfrom
gabriel-samfira:detect-multipart-mime
Closed

Detect multipart mime and ignore it#823
gabriel-samfira wants to merge 1 commit intoflatcar:mainfrom
gabriel-samfira:detect-multipart-mime

Conversation

@gabriel-samfira
Copy link
Member

Detect multipart mime user-data

This change makes ignition ignore user-data that is multipart mime.

How to use

Deploy a VM and add multipart mime userdata. If the VM boots, it works.

Testing done

Deployed a VM on OpenStack and set its userdata to a multipart mime userdata. The VM succeeded to boot.

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

@gabriel-samfira gabriel-samfira force-pushed the detect-multipart-mime branch 2 times, most recently from 3354f02 to a89ab8a Compare May 19, 2023 11:03
@gabriel-samfira gabriel-samfira force-pushed the detect-multipart-mime branch 2 times, most recently from 2b6b7b2 to 98034e4 Compare May 23, 2023 08:59
@gabriel-samfira
Copy link
Member Author

/build-image

1 similar comment
@gabriel-samfira
Copy link
Member Author

/build-image

@gabriel-samfira
Copy link
Member Author

/build-image

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
@gabriel-samfira gabriel-samfira force-pushed the detect-multipart-mime branch from 98034e4 to 1ea4811 Compare May 23, 2023 12:07
@gabriel-samfira
Copy link
Member Author

/build-image

@pothos
Copy link
Member

pothos commented May 30, 2023

Last time I checked 0019-detect-multipart-mime-and-ignore-it.patch didn't apply during the build. Is this solved already?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants