Skip to content

Conversation

@vpa1977
Copy link
Contributor

@vpa1977 vpa1977 commented Sep 29, 2025

Ping the @canonical/rocks team.


Description

This PR onboards chiselled tomcat images.

Related issues

#638


Picture of a cool rock:

@vpa1977 vpa1977 marked this pull request as ready for review September 30, 2025 00:23
@zhijie-yang zhijie-yang changed the title feat: onboard chiselled tomcat. feat: onboard chiselled tomcat Sep 30, 2025
- jdk-based rocks will be published in `tomcat-dev` rock and follow development rock specification.
- use <version>-<X.Y> version tags for "production" rock
@vpa1977 vpa1977 requested a review from cjdcordeiro October 2, 2025 19:19
commit: 59f3f98b37988ab3a07419098e755c892c44f93d
directory: tomcat/10-jre-21-24.04
release:
10-21-24.04:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO the channel is still weird.

Our convention is <swVersion>-<base>_<risk>. In this case, I find it really hard for a user to understand what's actually being delivered. Furthermore, it will make it really hard for things like the store to filter the rock by "version"...because - what is the version??

I ack the challenge though. No perfect, but what about 10jre21-24.04? I'd love to get the store folks opinion on this as well.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vpa1977 just checked with the store team, and the preferred approach would be:

10-jre21-24.04 (<swVersion>(-<platform>)-<base>).

One other thing you can do, to preserve an intuitive UX for users, is to still have 10-24.04, which uses the default jre in 24.04, and is the same artifact as 10-jre21-24.04.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOTE: This was resolved but not applied

Until we switch to SRUs for tomcat sources, use nil EOL to indicate that the images are essentially pre-release.
Removed nil values for end-of-life in Tomcat releases.
@vpa1977 vpa1977 requested a review from cjdcordeiro October 15, 2025 20:07
Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll also let @zhijie-yang and @alesancor1 handle the CI failure

@vpa1977
Copy link
Contributor Author

vpa1977 commented Oct 17, 2025

Closing PR - tomcat is an universe package and will be provided as a pro rock.

@vpa1977 vpa1977 closed this Oct 17, 2025
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.

2 participants