Skip to content

Conversation

@jnmt
Copy link
Collaborator

@jnmt jnmt commented Feb 3, 2025

Description

This PR adds release workflows for ScalarDL Ledger.

Related issues and/or PRs

N/A

Changes made

  • Add release workflows
  • Add settings to publish ScalarDL Ledger in Maven

Checklist

  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation to reflect the changes.
  • I have considered whether similar issues could occur in other products, components, or modules if this PR is for bug fixes.
  • Any remaining open issues linked to this PR are documented and up-to-date (Jira, GitHub, etc.).
  • Tests (unit, integration, etc.) have been added for the changes. I will check it after merging this.
  • My changes generate no new warnings.
  • Any dependent changes in other PRs have been merged and published.

Additional notes (optional)

N/A

@jnmt jnmt self-assigned this Feb 3, 2025
Comment on lines +7 to +8
- "[0-9]+"
- "[0-9]+.[0-9]+"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We will release SNAPSHOT versions of release and support branches to properly refer them from each release and support branches from the scalar repository.

Comment on lines +6 to +7
mavenJava(MavenPublication) {
artifactId = 'scalardl-ledger'
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@feeblefakie Not sure, but do we need to do something to release a new artifact in Maven?

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think so, but we'll see...

Comment on lines +51 to +52
docker push ghcr.io/scalar-labs/scalardl-ledger:${{ steps.version.outputs.version }}
docker push ghcr.io/scalar-labs/scalardl-client:${{ steps.version.outputs.version }}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We will release Docker images of the community edition with the new name "scalardl-xx" while keeping the old ones until we can confirm they are no longer referred.

BTW, do we really need to release the client image? Let me know your opinion or use cases if you know.

Copy link
Contributor

Choose a reason for hiding this comment

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

BTW, do we really need to release the client image? Let me know your opinion or use cases if you know.

At least, I think we are using ghcr.io/scalar-labs/scalar-client in scalardl-samples.
https://github.com/scalar-labs/scalardl-samples/blob/v3.10.0/docker-compose-auditor.yml#L23

However, the scalar-client is already public. So, I think we need to discuss which we should use scalar-client or salardl-client in the scalardl-samples. Let's discuss it tomorrow meeting!

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, sorry. I misunderstood. It's a SNAPSHOT image.

I think we don't use the scalar-client of the SNAPSHOT version anywhere.

If it includes ScalarDL CLI, some users or developers might want to use it for the testing of the main branch. But, I don't come up with some concrete use cases other than that, at the moment.

Copy link
Contributor

@feeblefakie feeblefakie left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

Comment on lines +6 to +7
mavenJava(MavenPublication) {
artifactId = 'scalardl-ledger'
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think so, but we'll see...

Copy link
Contributor

@kota2and3kan kota2and3kan left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

Comment on lines +51 to +52
docker push ghcr.io/scalar-labs/scalardl-ledger:${{ steps.version.outputs.version }}
docker push ghcr.io/scalar-labs/scalardl-client:${{ steps.version.outputs.version }}
Copy link
Contributor

Choose a reason for hiding this comment

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

BTW, do we really need to release the client image? Let me know your opinion or use cases if you know.

At least, I think we are using ghcr.io/scalar-labs/scalar-client in scalardl-samples.
https://github.com/scalar-labs/scalardl-samples/blob/v3.10.0/docker-compose-auditor.yml#L23

However, the scalar-client is already public. So, I think we need to discuss which we should use scalar-client or salardl-client in the scalardl-samples. Let's discuss it tomorrow meeting!

Comment on lines +51 to +52
docker push ghcr.io/scalar-labs/scalardl-ledger:${{ steps.version.outputs.version }}
docker push ghcr.io/scalar-labs/scalardl-client:${{ steps.version.outputs.version }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, sorry. I misunderstood. It's a SNAPSHOT image.

I think we don't use the scalar-client of the SNAPSHOT version anywhere.

If it includes ScalarDL CLI, some users or developers might want to use it for the testing of the main branch. But, I don't come up with some concrete use cases other than that, at the moment.

@jnmt jnmt merged commit ea744ea into master Feb 5, 2025
5 checks passed
@jnmt jnmt deleted the add-release-workflows branch February 5, 2025 10:36
jnmt added a commit that referenced this pull request Feb 6, 2025
jnmt added a commit that referenced this pull request Feb 6, 2025
jnmt added a commit that referenced this pull request Feb 6, 2025
jnmt added a commit that referenced this pull request Feb 6, 2025
@jnmt jnmt mentioned this pull request Mar 6, 2025
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants