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

added stage to debug gorelease failure #940

Merged
merged 4 commits into from
Apr 2, 2021
Merged

added stage to debug gorelease failure #940

merged 4 commits into from
Apr 2, 2021

Conversation

akankshakumari393
Copy link
Contributor

@akankshakumari393 akankshakumari393 commented Mar 26, 2021

Change Overview

This PR is to add a test job in kanister to build all tools images before the release. This will let us avoid image build failures when doing a kanister release.

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

Issues

  • #K10-6428

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

Comment on lines -130 to -131
archives:
- allow_different_binary_count: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this change is required @A-kanksh-a ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had added this as a part of #840, as I was getting different binary counts issue on the local machine while adding kafka-adobe-s3-connector related image to goreleaser.

After adding the command make release-snapshot with field allow_different_binary_count set to true caused issue.

And also, Kanister CI environment is not having different binary counts issue for kafka-adobe-s3-connector related image, so that flag is not needed.

@pavannd1 pavannd1 added the kueue label Apr 2, 2021
@mergify mergify bot merged commit a25ae11 into master Apr 2, 2021
@mergify mergify bot deleted the build-tool-image branch April 2, 2021 20:56
chaitanya-baraskar pushed a commit to chaitanya-baraskar/kanister that referenced this pull request Apr 5, 2021
* added stage to debug gorelease failure

* added a newline at the end

* removed archive

Co-authored-by: Pavan Navarathna <pavan@kasten.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants