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

Update config-to-label transform to include BALENA_* vars #848

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pipex
Copy link
Contributor

@pipex pipex commented Nov 12, 2021

Documentation mentions BALENA_SUPERVISOR_UPDATE_STRATEGY config var
as a way to set the update strategy, but this var has no effect as there
is no transform setup for it (there is only one for for RESIN_ prefix).

Change-type: patch
Closes: #735

@pipex
Copy link
Contributor Author

pipex commented Nov 15, 2021

@balena-ci retest

@pipex
Copy link
Contributor Author

pipex commented Nov 16, 2021

@balena-ci rebase

@ghost ghost force-pushed the update-strategy-vars branch from a39ba76 to 46262a9 Compare November 16, 2021 14:08
@pipex pipex requested a review from Page- November 16, 2021 18:29
@pipex
Copy link
Contributor Author

pipex commented Nov 16, 2021

@balena-ci rebase

@ghost ghost force-pushed the update-strategy-vars branch from 46262a9 to c50ba2e Compare November 16, 2021 18:29
@pipex
Copy link
Contributor Author

pipex commented Nov 19, 2021

@balena-ci rebase

[Documentation](https://www.balena.io/docs/learn/deploy/release-strategy/update-strategies/#controlling-the-update-strategy)
mentions `BALENA_SUPERVISOR_UPDATE_STRATEGY` config var
as a way to set the update strategy, but this var has no effect as there
is no transform setup for it (there is only one for for `RESIN_` prefix).

Change-type: patch
Closes: #735
@klutchell
Copy link
Contributor

@pipex anything blocking this, just a code review?

@pipex
Copy link
Contributor Author

pipex commented Apr 25, 2022

Not that I know of @klutchell, is just missing a code review. I brought it up in a thread in FD here https://www.flowdock.com/app/rulemotion/r-product/threads/6lvSr_BMwEH9iw6pXX_t4qG4ACY, but I have not pushed more on this change

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.

update strategy and handover timeout not being detect with BALENA_ prefix
2 participants