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

[NU-1790] Add sbt task to publish 1.16 compatibility jars #63

Open
wants to merge 16 commits into
base: staging
Choose a base branch
from

Conversation

mslabek
Copy link
Member

@mslabek mslabek commented Sep 25, 2024

No description provided.

@mslabek mslabek force-pushed the NU-1790-add-sbt-task-publish-flink116-kafka-components branch 3 times, most recently from 6f08b3c to 34db039 Compare September 26, 2024 13:43
@mslabek mslabek marked this pull request as ready for review September 26, 2024 13:44
@mslabek mslabek changed the title [NU-1790] Add sbt task to publish Kafka 1.16 components [NU-1790] Add sbt task to publish 1.16 compatibility jars Sep 26, 2024
@mslabek mslabek force-pushed the NU-1790-add-sbt-task-publish-flink116-kafka-components branch 2 times, most recently from b499c3e to 17961ad Compare September 26, 2024 15:11
@mslabek mslabek force-pushed the NU-1790-add-sbt-task-publish-flink116-kafka-components branch from 17961ad to e480709 Compare October 1, 2024 09:08
build.sbt Outdated Show resolved Hide resolved
build.sbt Outdated Show resolved Hide resolved
build.sbt Outdated Show resolved Hide resolved
@mslabek mslabek force-pushed the NU-1790-add-sbt-task-publish-flink116-kafka-components branch from 4e6b989 to a31fe6b Compare October 2, 2024 15:03
Copy link
Member Author

@mslabek mslabek left a comment

Choose a reason for hiding this comment

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

Thanks for the review!

build.sbt Outdated Show resolved Hide resolved
build.sbt Outdated Show resolved Hide resolved
build.sbt Outdated Show resolved Hide resolved
@mslabek mslabek force-pushed the NU-1790-add-sbt-task-publish-flink116-kafka-components branch from a31fe6b to ae4b664 Compare October 2, 2024 15:07
build.sbt Outdated Show resolved Hide resolved
build.sbt Outdated Show resolved Hide resolved
build.sbt Outdated
val flink116V = "1.16.0"
val currentFlinkV = "1.19.1"
val sttpV = "3.8.11"
val kafkaV = "3.3.1"
val testContainersScalaV = "0.41.0"

ThisBuild / version := "0.1-SNAPSHOT"
ThisBuild / version := "1.0-nu1.18.0-SNAPSHOT"

// todo: for now we should regularly bump the version until we start publish single "latest" -SNAPSHOT version
val defaultNussknackerV = "1.18.0-staging-2024-09-24-20698-40fc17dbe-SNAPSHOT"
Copy link
Contributor

Choose a reason for hiding this comment

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

why "default"?

Copy link
Member Author

Choose a reason for hiding this comment

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

Its default because it can be overriden by env variable from CI when testing triggered from main repo. I've moved this into nussknackerV scope to make it more clear.

Copy link
Contributor

Choose a reason for hiding this comment

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

IMO Nu version should be hardcoded and every time it should be bumped manually (and the change should be reflected with a commit). Like we did in other projects that use Nu as a dependency.

Copy link
Member Author

@mslabek mslabek Oct 3, 2024

Choose a reason for hiding this comment

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

But then triggering tests from core repo would require pushing a commit to nussknacker-flink-compatibility:staging branch with the tag of the released snapshot image instead of triggering an action? Or do you have another solution in mind?

Copy link
Contributor

Choose a reason for hiding this comment

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

I was thinking about the same solution we have in Quickstart and other Nu-related repos:

  1. there is an instruction in the Release Management guide to bump the project
  2. there is a master branch with the latest released Nu version
  3. there is a staging branch with the snapshot version of the Nu version
  4. Nu core pipeline triggers this pipeline (for the staging branch) to check if the newest staging version of Nu doesn't cause any problems with this repo

build.sbt Outdated Show resolved Hide resolved
build.sbt Outdated Show resolved Hide resolved
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