Skip to content

Conversation

mrsimonemms
Copy link
Contributor

@mrsimonemms mrsimonemms commented Mar 17, 2022

Description

This is derived from #8840 and the initial work on the recommended items

Related Issue(s)

Fixes #8846 - memory check
Fixes #8856 - default storage class

How to test

Deploy via KOTS and run preflights

Release Notes

[KOTS]: add recommended preflights

Documentation

@mrsimonemms mrsimonemms marked this pull request as ready for review March 17, 2022 11:08
@mrsimonemms mrsimonemms requested a review from a team March 17, 2022 11:08
@mrsimonemms mrsimonemms changed the title WIP: [KOTS]: add recommended preflights [KOTS]: add recommended preflights Mar 17, 2022
@github-actions github-actions bot added the team: delivery Issue belongs to the self-hosted team label Mar 17, 2022
@mrsimonemms mrsimonemms marked this pull request as draft March 17, 2022 13:34
@mrsimonemms mrsimonemms force-pushed the sje/kots-preflights branch from 3edd999 to cc331c6 Compare March 17, 2022 13:37
@mrsimonemms mrsimonemms marked this pull request as ready for review March 17, 2022 16:37
Copy link
Contributor

@nandajavarma nandajavarma left a comment

Choose a reason for hiding this comment

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

The changes look good to me. I was trying to test it out with the image built with this PR, but I still see the old preflights. I might be missing something. will get back.

@mrsimonemms
Copy link
Contributor Author

The changes look good to me. I was trying to test it out with the image built with this PR, but I still see the old preflights. I might be missing something. will get back.

It's not in the unstable channel. You'll need to run make and publish to your own dev channel

@mrsimonemms
Copy link
Contributor Author

@Pothulapati we do actually create PVCs for in cluster Docker registry and Minio

Pothulapati
Pothulapati previously approved these changes Mar 22, 2022
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.

Changes LGTM then!

@@ -84,8 +84,16 @@ spec:
message: There must be at least one node with the label "gitpod.io/workload_workspace_headless" and value "true"
- pass:
message: Node with label exists
- customResourceDefinition:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I moved this up so the memory/CPU checks are in the same section - seemed more logical that way

@mrsimonemms mrsimonemms requested a review from Pothulapati March 22, 2022 07:51
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.

:shipit:

@roboquat roboquat merged commit 89ac56f into main Mar 22, 2022
@roboquat roboquat deleted the sje/kots-preflights branch March 22, 2022 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note size/M team: delivery Issue belongs to the self-hosted team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a check for a default StorageClass Add memory check
4 participants