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

tests: deploy knative addon only when necessary #3316

Closed
wants to merge 1 commit into from

Conversation

pmalek
Copy link
Member

@pmalek pmalek commented Jan 3, 2023

What this PR does / why we need it:

This PR makes it so that knative tests that require knative addon to be installed install it themselves, whenever needed. This makes all the other tests to run without knative addon, faster and it also makes the knative addon not install itself when running tests manually with -run Test.... flag.

@pmalek pmalek added area/knative area/maintenance Cleanup, refactoring, and other maintenance improvements that don't change functionality. labels Jan 3, 2023
@pmalek pmalek self-assigned this Jan 3, 2023
@pmalek pmalek force-pushed the deploy-knative-addon-only-when-necessary branch 2 times, most recently from f0e5d83 to f35702c Compare January 4, 2023 11:04
@pmalek pmalek force-pushed the deploy-knative-addon-only-when-necessary branch from f35702c to ef30742 Compare January 4, 2023 13:58
@pull-request-size pull-request-size bot added size/M and removed size/S labels Jan 4, 2023
@pmalek
Copy link
Member Author

pmalek commented Jan 10, 2023

This seems to be failing because we don't start controllers for which we don't have CRDs installed (feature) :)

time="2023-01-10T11:22:01+01:00" level=info msg="disabling the 'ingresses' controller due to missing CRD installation" logger=controllers.crdCondition

Closing. We can revisit this when we figure #3029 out

@pmalek pmalek closed this Jan 10, 2023
@pmalek pmalek deleted the deploy-knative-addon-only-when-necessary branch March 4, 2023 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/knative area/maintenance Cleanup, refactoring, and other maintenance improvements that don't change functionality. area/tests size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant