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

Disable dev variant substitution as they have merged #492

Closed
wants to merge 1 commit into from

Conversation

klutchell
Copy link
Collaborator

Change-type: patch
Signed-off-by: Kyle Harding kyle@balena.io
See: balena-os/meta-balena#2227

Although we are still publishing both .dev and .prod variants for now, they are actually the same image going forward.

Opening as DRAFT for a couple reasons:

  1. this change may be breaking, not sure yet
  2. this helper may be in the process of being moved to another repo entirely

Change-type: patch
Signed-off-by: Kyle Harding <kyle@balena.io>
@rcooke-warwick
Copy link
Contributor

hey @klutchell good thinking here but I don't think I've seen anyone ever use this helper - so in the cleanup with removing helpers from this repo and into an npm package, I've actually removed it: https://github.com/balena-os/test-helpers

@klutchell
Copy link
Collaborator Author

@rcooke-warwick I think it's being used in the hup suite, which is how I ended up here.
https://github.com/balena-os/meta-balena/blob/master/tests/suites/hup/suite.js#L287

@vipulgupta2048
Copy link
Member

@rcooke-warwick I created this helper to facilitate downloading OS images in a standardized way. It is in use. I am surprised that the hup suite run didn't fail when the helper was removed.

@vipulgupta2048
Copy link
Member

But, the PR sounds good.

@klutchell
Copy link
Collaborator Author

@balena-ci rebase

@ghost ghost force-pushed the klutchell/remove-variants branch from 509a7df to 96c1c3e Compare December 4, 2021 14:27
@klutchell
Copy link
Collaborator Author

@balena-ci rebase

@ghost ghost force-pushed the klutchell/remove-variants branch from 96c1c3e to bc6eaa9 Compare January 11, 2022 16:09
@klutchell
Copy link
Collaborator Author

Superseded by #614

@klutchell klutchell closed this Jan 18, 2022
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.

None yet

3 participants