-
Notifications
You must be signed in to change notification settings - Fork 15
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 Jenkinsfile to use Fuji style #26
update Jenkinsfile to use Fuji style #26
Conversation
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.
Major refactor of Jenkinsfile - thanks @ernestojeda
Working sandbox job here: |
Signed-off-by: Ernesto Ojeda <[email protected]>
a24abc0
to
29238f3
Compare
@iain-anderson please review |
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.
Version should be 1.0.0 at the moment
@lranjbar please weigh in here as this service although versioned independently was slated for FUJI, not EDINBURGH release so I"m not following the need to version as 1.0.0 as @iain-anderson suggests. |
The service as it stands builds against the Edinburgh version of the SDK and advertises itself as version 1.0.0. If we are doing the latter, this PR is fine |
Signed-off-by: Ernesto Ojeda <[email protected]>
As Fuji is just a minor release of the 1.x, there's no need to align the version of this service with Fuji's version. The rules for application and device service is that the major version needs to align with the compatible major version of EdgeX, but that minor and/or patch releases aren't tied to specific minor or patch releases of EdgeX. One other question, it looks like the build environment has been changed from Alpine to CentOS, are we sure that there aren't any libc compatibility issues building the artifacts on CentOS and running in an Alpine-based container? |
I talked to Iain on slack and to verify for device-bacnet-c we are going to release a 1.0.0 version for Edinburgh first. After this PR is merged we need to cut the edinburgh branch. Changes after this release will be to verify the service is compatible with the Fuji release. Due to timing this will happen after the main Fuji release. |
Isn't it that we are building in an Alpine container running on a Centos host? |
@iain-anderson that's correct. this builds two images but uses two different hosts (VMs) - one Centos x-86 and the other Ubuntu-ARM64 |
Yes, we are actually building an x86_64 image and an arm64 image like we do for all the other projects. Here are the agent declarations: |
Signed-off-by: Ernesto Ojeda [email protected]