ci: adjust build arguments#2772
Conversation
The build arguments `VCS_REF` and `VCS_VER` were renamed and given proper values according to their names. 1. `VCS_REVISION` holds the current SHA sum of the (git) HEAD pointer 2. `VCS_VERSION` now holds the contents of the `VERSION` file, i.e. a semver version tag (one can now inspect the image and find a proper version tag in the `org.opencontainers.image.version` label) The build arguments were given defaults in order to allow the `generic_build` and `generic_test` workflows to omit them (as they are not need there anyways). When publishing images, this is fina as the cache will rebuild almost all of the image except the last few layers which are `LABEL`s anyways.
|
@polarathene just had a quick glance at CI, found this: Do you think we need to worry here? Everything seems to be working. This may only occur on the first run? |
It looks fine, no cache was restored for the build job, so when it tries to look for the cache to restore during the "build image step" it has nothing to restore, and emits a warning about it, in case you expected cache to be there. We see that it uploads the cache afterwards fine, and the test job restores that properly. However, I don't think the job was meant to upload a new cache to Build job:
Test job:
|
|
@polarathene I will have a look at the cache issue next week :) |
|
@georglauterbach alright, just keep in mind that any time a cache doesn't exist for the expected cache key, the fallback (restore key) is now Until it expires, a fix may not fallback correctly, but at least for the test job that shouldn't be an issue, the build job is always meant to make a valid cache key for a direct match. |
Description
The build arguments
VCS_REFandVCS_VERwere renamed and given proper values according to their names.VCS_REVISIONholds the current SHA sum of the (git) HEAD pointerVCS_VERSIONnow holds the contents of theVERSIONfile, i.e. a semver version tag (one can now inspect the image and find a proper version tag in theorg.opencontainers.image.versionlabel)The build arguments were given defaults in order to allow the
generic_buildandgeneric_testworkflows to omit them (as they are not need there anyways). When publishing images, this is fina as the cache will rebuild almost all of the image except the last few layers which areLABELs anyways.Type of change
Checklist:
docs/)