-
Notifications
You must be signed in to change notification settings - Fork 54
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
Java: Create Workflow to deploy artifacts for all platforms mentioned in build.matrix.json #2285
Conversation
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
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.
You can revert files irrelevant to your changes by getting the file from main using https://www.freecodecamp.org/news/git-checkout-file-from-another-branch/
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
.github/workflows/java-cd.yml
Outdated
|
||
runs-on: ${{ matrix.host.RUNNER }} | ||
steps: | ||
- name: Start Redis server |
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.
Why redis, not valkey? Why you don't reuse install-valkey
action?
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.
i think it was because the procedure to install redis is easier than valkey. i can try out install-valkey
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.
Is it really that different? Valkey is just a fork of Redis.
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.
actually, valkey doesn't have an installer package and you need to install from source.
but the install-valkey workflow is available to do that for us.
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
@@ -12,7 +13,7 @@ dependencies { | |||
def releaseVersion = System.getenv("GLIDE_RELEASE_VERSION"); | |||
|
|||
if (releaseVersion) { | |||
implementation "io.valkey:valkey-glide:" + releaseVersion | |||
implementation "io.valkey:valkey-glide:" + releaseVersion + ":${osdetector.classifier}" |
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.
🌟
test-deployment-on-all-architectures: | ||
needs: [set-release-version, load-platform-matrix, publish-to-maven-central-deployment] | ||
env: | ||
JAVA_VERSION: "11" |
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.
ok - fine. good enough.
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.
Please update PR description and title and merge
All comments were addressed
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.
Add to changelog: to the operational changes section
Signed-off-by: Chloe Yip <[email protected]>
… in build.matrix.json (valkey-io#2285) Signed-off-by: Chloe Yip <[email protected]>
… in build.matrix.json (valkey-io#2285) Signed-off-by: Chloe Yip <[email protected]>
This includes:
Testing against 1.0.2-rc1