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

Add the spec.selector field in the time-logger deployment #3121

Merged
merged 3 commits into from
Sep 22, 2024

Conversation

anishbista60
Copy link
Contributor

@anishbista60 anishbista60 commented Sep 16, 2024

Change Overview

Added the spec.selector field in the time-logger deployment example in kanister new deployment.

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test
  • 🏗️ Build

Issues

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

Signed-off-by: Anish Bista <anishbista053@gmail.com>
@anishbista60 anishbista60 changed the title Add the spec.selector filed in the time-logger deployment Add the spec.selector field in the time-logger deployment Sep 16, 2024
@anishbista60
Copy link
Contributor Author

@shuguet could you please take a look here ?

@PrasadG193
Copy link
Contributor

Looks good, thanks for fixing this

@anishbista60
Copy link
Contributor Author

@PrasadG193 do you have right to merge it ?

@anishbista60
Copy link
Contributor Author

mergify doesn't have enough permission.

@julio-lopez julio-lopez removed the kueue label Sep 20, 2024
@julio-lopez julio-lopez merged commit 6eef775 into kanisterio:master Sep 22, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing spec.selector field in time-logger deployment manifest example in kanister new documentation
3 participants