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

docs: Rework blueprints to separate DB tools image and kanister-tools image #3267

Closed
wants to merge 6 commits into from

Conversation

hairyhum
Copy link
Contributor

Change Overview

Separation of images allows kanister maintainters to build less images in kanister release process and allows the users of those blueprints to always use up to data databse images.

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

  • fixes #issue-number

Test Plan

TODO: All updated blueprints need to be tested

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

… image

Separation of images allows us to build less images in kanister release process
and allows the users of those blueprints to always use up to data databse images.

Signed-off-by: Daniil Fedotov <[email protected]>
@viveksinghggits
Copy link
Contributor

@hairyhum are you looking for reviews in this PR? I am asking this because the PR is still in draft state.

@hairyhum
Copy link
Contributor Author

hairyhum commented Dec 9, 2024

are you looking for reviews in this PR? I am asking this because the PR is still in draft state.

@viveksinghggits
It's in the test stage and I'm making changes to make the tests work, which will be split into different PRs, but if you have any feedback - feel free to post a comment here.

@hairyhum
Copy link
Contributor Author

Split this PR into #3285 #3284 #3286 #3287

@hairyhum hairyhum closed this Dec 10, 2024
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.

2 participants