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

[installer]: add in ability to post-process #11391

Merged
merged 2 commits into from
Jul 19, 2022
Merged

Conversation

mrsimonemms
Copy link
Contributor

@mrsimonemms mrsimonemms commented Jul 14, 2022

Description

Adds the ability to post-process the Installer's YAML natively in the Golang binary. This extends the YQ library, so any valid YQ expression can be used to amend the generated YAML.

It also removes the status object from both Network Policies and Stateful Sets, both of which are problematic when supporting multiple versions of Kubernetes.

This also removes the problems associated with upgrades to the Kubernetes libraries (eg #11003) and allows for #11177 to be merged.

How to test

Run the Installer, there should be no status in any NetworkPolicy or the OpenVSXProxy StatefulSet resources.

Release Notes

[installer]: add in ability to post-process the generated YAML

Documentation

Werft options:

  • /werft with-preview

@mrsimonemms mrsimonemms force-pushed the sje/installer-post-process branch 2 times, most recently from 1f42764 to 2f7014c Compare July 14, 2022 21:53
@mrsimonemms mrsimonemms marked this pull request as ready for review July 15, 2022 09:05
@mrsimonemms mrsimonemms requested a review from a team July 15, 2022 09:05
@github-actions github-actions bot added the team: delivery Issue belongs to the self-hosted team label Jul 15, 2022
@mrsimonemms mrsimonemms force-pushed the sje/installer-post-process branch from 2f7014c to 0005267 Compare July 15, 2022 15:36
@mrsimonemms mrsimonemms requested a review from a team July 15, 2022 15:36
@mrsimonemms mrsimonemms force-pushed the sje/installer-post-process branch from 0005267 to a213fbb Compare July 16, 2022 15:41
@mrsimonemms mrsimonemms removed the request for review from a team July 16, 2022 15:41
@mrsimonemms mrsimonemms force-pushed the sje/installer-post-process branch from a213fbb to 35b9060 Compare July 18, 2022 13:44
@mrsimonemms mrsimonemms force-pushed the sje/installer-post-process branch from 35b9060 to d6dad48 Compare July 19, 2022 08:37
This removes the "status" block on the root, which breaks older versions
of Kubernetes
@mrsimonemms mrsimonemms force-pushed the sje/installer-post-process branch from d6dad48 to 4e2d985 Compare July 19, 2022 08:40
Copy link
Contributor

@Pothulapati Pothulapati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note size/L team: delivery Issue belongs to the self-hosted team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants