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

Revert "feat(kubectl): Extract interface for KubectlJobExecutor and u… #6092

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

DanielaS12
Copy link
Contributor

…se it. (#6076)"

This reverts commit bb04878.

This commit is not needed anymore, for easier way to use and interact with the kubernetes module, this needs to be reverted.

Tested locally, the app runs successfully.
image

@spinnakerbot
Copy link
Contributor

The following commits need their title changed:

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

@ovidiupopa07 ovidiupopa07 added the ready to merge Approved and ready for a merge label Nov 14, 2023
@mergify mergify bot added the auto merged Merged automatically by a bot label Nov 14, 2023
@mergify mergify bot merged commit c4d1827 into spinnaker:master Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Approved and ready for a merge target-release/1.33
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants