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

feat: add flag to enable LB SVC for kong admin in CLI #245

Merged
merged 1 commit into from
Mar 22, 2022

Conversation

shaneutt
Copy link
Contributor

This is just something I do manually with a fair amount of frequency so I figured I would add it to the CLI.

In time we do need to change how the CLI is structured so that flags belong to the addon component instead of being general to environment creation, but for the moment this is just a quick add that provides a lot of value for running tests.

@shaneutt shaneutt self-assigned this Mar 22, 2022
@shaneutt shaneutt temporarily deployed to integration-tests March 22, 2022 14:27 Inactive
@shaneutt shaneutt marked this pull request as ready for review March 22, 2022 14:37
@shaneutt shaneutt requested a review from a team as a code owner March 22, 2022 14:37
@shaneutt shaneutt enabled auto-merge (squash) March 22, 2022 14:37
@codecov-commenter
Copy link

Codecov Report

Merging #245 (ff17e16) into main (56efc53) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #245   +/-   ##
=======================================
  Coverage   11.75%   11.75%           
=======================================
  Files           5        5           
  Lines         604      604           
=======================================
  Hits           71       71           
  Misses        528      528           
  Partials        5        5           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 56efc53...ff17e16. Read the comment docs.

@shaneutt shaneutt merged commit 96aedc3 into main Mar 22, 2022
@shaneutt shaneutt deleted the shaneutt/flag-admin-svc-lb-in-cli branch March 22, 2022 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants