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

remove consul-k8s submodule #21622

Merged
merged 1 commit into from
Aug 20, 2024
Merged

remove consul-k8s submodule #21622

merged 1 commit into from
Aug 20, 2024

Conversation

jm96441n
Copy link
Member

Description

Remove consul-k8s submodule accidentally introduced in #21279

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@jm96441n jm96441n added pr/no-changelog PR does not need a corresponding .changelog entry backport/1.19 This release series is longer active on CE, use backport/ent/1.19 labels Aug 20, 2024
@jm96441n
Copy link
Member Author

@boruszak just wanted to check with you that this submodule wasn't necessary since it was added in your PR

@jm96441n jm96441n enabled auto-merge (squash) August 20, 2024 15:13
@jm96441n jm96441n merged commit 9d06fc3 into main Aug 20, 2024
111 checks passed
@jm96441n jm96441n deleted the remove-submodule branch August 20, 2024 15:18
philrenaud pushed a commit that referenced this pull request Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.19 This release series is longer active on CE, use backport/ent/1.19 pr/no-changelog PR does not need a corresponding .changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants