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

Fixed BOTKUBE_VERSION #483

Merged
merged 2 commits into from
May 21, 2021
Merged

Fixed BOTKUBE_VERSION #483

merged 2 commits into from
May 21, 2021

Conversation

pierluigilenoci
Copy link
Contributor

@pierluigilenoci pierluigilenoci commented May 11, 2021

ISSUE TYPE
  • Bug fix Pull Request
SUMMARY

The software notifies on the channel if you are using the updated version of BotKube or not. To do this, use the BOTKUBE_VERSION variable which is set up with the .Chart.Version value which does not necessarily correspond to the version of the software running on the cluster. The actual version of the software that runs is the one that comes from .Values.image.tag.

Screenshot 2021-05-11 at 09 51 39

@sanketsudake sanketsudake requested a review from PrasadG193 May 11, 2021 07:59
@PrasadG193
Copy link
Collaborator

We need to revisit version logic to use build-time variables. But this can be done as a separate task.

@pierluigilenoci
Copy link
Contributor Author

@PrasadG193 all build failed? Why?

@pierluigilenoci
Copy link
Contributor Author

@PrasadG193 what can we do to push forward?

@PrasadG193
Copy link
Collaborator

@pierluigilenoci looks like one of the dep breaking builds. Let me fix that.

@mergify mergify bot merged commit 5c40418 into kubeshop:develop May 21, 2021
@pierluigilenoci pierluigilenoci deleted the botkube_version branch May 21, 2021 13:44
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