-
Notifications
You must be signed in to change notification settings - Fork 21
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
Introduce a branch_version concept in BCI #1830
Conversation
Created a staging project on OBS for 5: home:defolos:BCI:Staging:SLE-15-SP5:5-1830 |
Created a staging project on OBS for 7: home:defolos:BCI:Staging:SLE-15-SP7:7-1830 Build ResultsRepository
Repository
Repository
Repository
Repository
Repository
Repository
Repository
Build failed ❌ To run BCI-tests against this PR, use the following command: OS_VERSION=15.7 TARGET=custom BASEURL=registry.opensuse.org/home/defolos/bci/staging/sle-15-sp7/7-1830/ tox -- -n auto The following images can be pulled from the staging project:
|
Created a staging project on OBS for Tumbleweed: home:defolos:BCI:Staging:Tumbleweed:Tumbleweed-1830 Build ResultsRepository
Repository
Repository
Repository
Build succeeded ✅ To run BCI-tests against this PR, use the following command: OS_VERSION=tumbleweed TARGET=custom BASEURL=registry.opensuse.org/home/defolos/bci/staging/tumbleweed/tumbleweed-1830/ tox -- -n auto The following images can be pulled from the staging project:
|
Created a staging project on OBS for 6: home:defolos:BCI:Staging:SLE-15-SP6:6-1830 Build ResultsRepository
Repository
Repository
Repository
Repository
Repository
Repository
Repository
Build failed ❌ To run BCI-tests against this PR, use the following command: OS_VERSION=15.6 TARGET=custom BASEURL=registry.opensuse.org/home/defolos/bci/staging/sle-15-sp6/6-1830/ tox -- -n auto The following images can be pulled from the staging project:
|
b194f1a
to
36b84fd
Compare
5a4cbeb
to
57a54bf
Compare
57a54bf
to
e3737f9
Compare
(ready for review but since I added tests it will require a rebase again once #1828 is merged) |
e3737f9
to
cc51e71
Compare
cc51e71
to
6a922a9
Compare
6a922a9
to
77a78df
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but there's a conflict to be resolved.
77a78df
to
cce5677
Compare
cce5677
to
d0973ac
Compare
Now that we have build_flavors, we can add branch_versions, describing the full hierachy: Each container can have a single `name`, one or many `branch_versions` and each of them one or many `build_flavors`. this means our naming should become {name}-{branch_version}-{build_flavor} Introduce this for the AppCollection containers first. This should eventually replace tag_version, which is a broken concept as it mixes namings, versions and flavors into and hence makes everything convoluted. This commit mixes also a hook in Registry to use version, rather than tag_version for the build_version, which is a must-have for the AppCollection integration. Add tests for ApplicationStackContainers in unit test, which was entirely untested so far. Removal of the deprecated tag_version will happen in a later followup.
d0973ac
to
2762bf3
Compare
Now that we have build_flavors, we can add branch_versions, describing the full hierachy:
Each container can have a single
name
, one or manybranch_versions
and each of them one or manybuild_flavors
.this means our naming should become
{name}-{branch_version}-{build_flavor}
Introduce this for the AppCollection containers first. This should eventually replace tag_version, which is a broken concept as it mixes namings, versions and flavors into and hence makes everything convoluted.
This commit mixes also a hook in Registry to use version, rather than tag_version for the build_version, which is a must-have for the AppCollection integration.
Removal of the deprecated tag_version will happen in a later followup.