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

use namespace kong for integration tests #4439

Merged
merged 2 commits into from
Aug 9, 2023

Conversation

randmonkey
Copy link
Contributor

What this PR does / why we need it:

use namespace kong instead of kong-system for integration tests, and change the hard coded namespace kong-system to const.ControllerNamespace.

Which issue this PR fixes:

fixes #4431

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@randmonkey randmonkey requested a review from a team as a code owner August 1, 2023 03:49
@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.2% ⚠️

Comparison is base (7df5df2) 67.9% compared to head (048dd9c) 67.8%.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #4439     +/-   ##
=======================================
- Coverage   67.9%   67.8%   -0.2%     
=======================================
  Files        161     161             
  Lines      18817   18817             
=======================================
- Hits       12788   12764     -24     
- Misses      5270    5296     +26     
+ Partials     759     757      -2     

see 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@randmonkey randmonkey force-pushed the test/use_namespace_kong_integration branch from 59db41d to 59eb210 Compare August 1, 2023 03:57
@randmonkey randmonkey marked this pull request as draft August 1, 2023 03:57
@randmonkey
Copy link
Contributor Author

blocked by Kong/kubernetes-testing-framework#765

@randmonkey randmonkey force-pushed the test/use_namespace_kong_integration branch from 59eb210 to 020b873 Compare August 1, 2023 06:35
@pull-request-size pull-request-size bot added size/S and removed size/M labels Aug 1, 2023
@randmonkey randmonkey force-pushed the test/use_namespace_kong_integration branch from 020b873 to 17a7ad2 Compare August 8, 2023 09:55
@pull-request-size pull-request-size bot removed the size/S label Aug 8, 2023
@randmonkey randmonkey added size/S and removed blocked labels Aug 8, 2023
@randmonkey randmonkey removed the size/S label Aug 8, 2023
@randmonkey randmonkey force-pushed the test/use_namespace_kong_integration branch from 17a7ad2 to 048dd9c Compare August 9, 2023 07:21
@pull-request-size pull-request-size bot added size/S and removed size/M labels Aug 9, 2023
@team-k8s-bot
Copy link
Collaborator

Integration (targeted) tests with KIND-based cluster and latest images were started at https://github.com/Kong/kubernetes-ingress-controller/actions/runs/5806095248

@randmonkey randmonkey self-assigned this Aug 9, 2023
@randmonkey randmonkey marked this pull request as ready for review August 9, 2023 07:50
@randmonkey randmonkey merged commit 5a15bc7 into main Aug 9, 2023
@randmonkey randmonkey deleted the test/use_namespace_kong_integration branch August 9, 2023 15:32
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.

Use kong namespace in integration tests
3 participants