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

Release of new versions. #1933

Open
AndriMar opened this issue Jan 16, 2024 · 4 comments
Open

Release of new versions. #1933

AndriMar opened this issue Jan 16, 2024 · 4 comments
Labels
question Further information is requested

Comments

@AndriMar
Copy link

Hello,
I have a question regarding the release pipeline and how the versions are bumped.
I was using the 7.4.0 version and notice a bug that I reported and really happy about how quick that was fixed.

I notice you have a Release branch Release-7.4 and after the initial release, changes keep on coming in which is fine but the patch number is not bumped in dockerhub and 7.4.0 images is overwritten.

I was expecting to need to change the deployment of X-Rode and point to the 7.4.1 image.

This is not affecting me that much because I want to keep my images in a private ECR but for those that are pulling directly from dockerhub, this could have big implications getting unwanted patches when container goes down and comes back up.

Is this an oversight or some other reason the patch number is not bumped?

@raits raits added the question Further information is requested label Jan 16, 2024
@petkivim
Copy link
Contributor

Hi @AndriMar,

The patch number is bumped when there's a change in the X-Road application code and a new version of the X-Road application is released. In this case, the bug was in the Security Server Docker image and not in the application code. Therefore, the Docker image was updated without making changes to the application code and therefore, the version number wasn't updated.

Also, monthly security updates to image are provided without bumping the version number. The updates are released monthly on the 25th, with additional releases for critical vulnerabilities. The updates include the latest base image with up-to-date packages and security fixes. There are no changes to X-Road Security Server software and its packages inside.

@bensi94
Copy link

bensi94 commented Jan 16, 2024

Hi @petkivim,

I just want to weigh in and give my personal opinion on this matter.
I think you should really reconsider this decision it could have serious implications on production systems. It's very often the case that Docker images are used directly from Docker Hub and developers and system admins trust that the tags from trusted entities do not change. You have to note that with container images your software is not only your application code it includes the Docker images, and it should be really important that tagged images are not changed.

@petkivim
Copy link
Contributor

@bensi94 Thank you for your feedback and sharing the insights!

The challenge with the current version numbering is that it's the X-Road application version number and the same number is used on all the supported platforms (Docker, Ubuntu, RHEL). Therefore, the version number must be aligned between the platforms and it cannot be increased for one platform only. One potential solution is to introduce an additional Docker image version number that's used in the tags, e.g., niis/xroad-security-server-sidecar:7.4.0-0, niis/xroad-security-server-sidecar:7.4.0-1, niis/xroad-security-server-sidecar:7.4.0-0-slim, niis/xroad-security-server-sidecar:7.4.0-1-slim, etc. We'll consider this in the future.

@AndriMar
Copy link
Author

That sounds like a solution. 👍

For example we had a custom CMD command to fix this in the earlier version, later we deployed this at a customers infrastructure and had no idea that we did not need this because the fix was already deployed.
Like @bensi94 mention, we need to know what we have running in production.

Thank you for you time and answers and hope to see immutable docker tags in the coming future 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants