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

ci: adds missing-images automation #1234

Merged
merged 2 commits into from
May 10, 2023
Merged

Conversation

faiq
Copy link
Contributor

@faiq faiq commented May 9, 2023

What problem does this PR solve?:
updates hack/airgapped/missing-images.json in kommander repo from local values here for the nvidia operator

Which issue(s) does this PR fix?:

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


Checklist

  • If the PR adds a version bump, ensure there is no breaking change in Licensing model (or NA).
  • If a chart is changed or app configuration is significantly changed, the chart version is correctly incremented (so that apps are not automatically upgraded from a previous version of DKP).

@faiq faiq marked this pull request as ready for review May 9, 2023 21:36
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label May 9, 2023
@faiq faiq mentioned this pull request May 9, 2023
2 tasks
@d2iq-mergebot
Copy link
Contributor

This repo has @d2iq-mergebot integration. You can perform the following commands by submitting a comment. Submit a comment with content "@d2iq-mergebot help" to view more detailed help text and examples. Be sure the have a look at the mergebot documentation, too.For help using mergebot, please refer to the README file here: https://github.com/mesosphere/mergebot/blob/main/README.md
Enabled Mergebot commands:
@d2iq-mergebot test all
@d2iq-mergebot test
@d2iq-mergebot override-status
@d2iq-mergebot help
@d2iq-mergebot backport

@coveralls
Copy link

coveralls commented May 9, 2023

Pull Request Test Coverage Report for Build 4941455206

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 79.532%

Totals Coverage Status
Change from base Build 4940727984: 0.0%
Covered Lines: 136
Relevant Lines: 171

💛 - Coveralls

@faiq faiq force-pushed the faiq/nvidia-branch-automation branch from 66b9f8f to b8d9681 Compare May 10, 2023 20:40
@karishmagadiya karishmagadiya added the ok-to-test Signals mergebot that CI checks are ready to be kicked off label May 10, 2023
@takirala takirala merged commit 51cb62e into main May 10, 2023
8 checks passed
@takirala takirala deleted the faiq/nvidia-branch-automation branch May 10, 2023 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Signals mergebot that CI checks are ready to be kicked off ready-for-review size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
6 participants