NO-ISSUE: Make sure kie-tools is using the latest timestamped SNAPSHOT from Kogito (999-20240411-SNAPSHOT)

[yesamer]

Update all Kogito references to 999-20240411-SNAPSHOT

[tiagobento]

I noticed we have different dates for the images... Could we unify?

[yesamer]

@tiagobento At this time, the latest version of the image https://quay.io/repository/kiegroup/kogito-base-builder-nightly?tab=tags&tag=latest is 2024-04-05

[tiagobento]

@ricardozanini Any idea what could've happened with the nightly build of this image?

[ricardozanini]

@yesamer @tiagobento this image has been removed as part of the old operator archiving: apache/incubator-kie-kogito-images#1748

[ricardozanini]

If you want, I think I can help review these PRs related to images. Let me know!

[yesamer]

@tiagobento @thiagoelg That means we need a replacement for that... Should I open a ticket?

[ricardozanini]

@yesamer why do you need a replacement? That image won't be used anywhere as far as I know.

[tiagobento]

@ricardozanini It is used by Serverless Logic Web Tools, it seems like... cc @caponetto

[paulovmr]

@ricardozanini @tiagobento yes this image was the base for the web tools builder and dev-mode images. @ricardozanini could it be restored?

[ricardozanini]

@paulovmr yes, we can. I can't do it today, if some of you have some room, here's the PR where I removed it: https://github.com/apache/incubator-kie-kogito-images/pull/1748/files

[ricardozanini]

@paulovmr @caponetto honestly, I saw the image now and I'm not seeing anything that it's not on openjdk-17 images. Why not use it directly?

[yesamer]

@rodrigonull Great!!!! Are you launching the job for all our repos? jit-executor as well? Thank you!

[rodrigonull]

@yesamer yes, my plan is to trigger all jobs including the JIT executor.

[tiagobento]

@yesamer For images, please change the tags to 999-YYYYMMDD instead of main-*, as the first comes from the weekly jobs (dependencies are also timestamped SNAPSHOTs) and the second is from daily jobs (dependencies are 999-SNAPSHOT)

[yesamer]

@tiagobento Done!

[gitgabrio]

Hi @yesamer
Out of curiosity: would not be possible to store that variable in one single place, and maybe refer to it wherever is needed ?

[tiagobento]

@gitgabrio That's the idea behind https://github.com/apache/incubator-kie-tools/blob/main/packages/root-env/env/index.js#L53-L60, but it's not always simple to do it, especially when the format changes slightly :)

[yesamer]

@gitgabrio I guess @tiagobento can give you a better answer than me, anyway my feeling is that is it possible to remove some of those places, and the tooling team is already committed to that. In this PR I removed one place :)

[yesamer]

@tiagobento Can I merge it? :)

[tiagobento]

@yesamer There's only one approve :( We need 2.

In the mean time...... could we remove KN_PLUGIN_WORKFLOW__quarkusVersion and KN_PLUGIN_WORKFLOW__kogitoVersion? We can use the one from root-env instead of having two variables dedicated to the kn-plugin-workflow.. and we @paulovmr's blessing :)

[yesamer]

@tiagobento Right.

So should I just remove KN_PLUGIN_WORKFLOW__kogitoVersion and replace it with kogitoRuntime.version, correct?

[tiagobento]

@yesamer Yes!

[tiagobento]

@yesamer Looks like this one can be removed too! SERVERLESS_LOGIC_WEB_TOOLS__kogitoVersion cc @paulovmr

[tiagobento]

@yesamer And this one too! SERVERLESS_LOGIC_WEB_TOOLS__quarkusPlatformVersion.

This is the last comment I promise 😅

[tiagobento]

I bet @gitgabrio is thrilled with us removing env vars, though :)

[yesamer]

BINGO!

[yesamer]

Done! @tiagobento @gitgabrio @kbowers-ibm