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

Start exporting Java SDK #9

Closed
wants to merge 1 commit into from
Closed

Conversation

kylepl
Copy link

@kylepl kylepl commented Nov 1, 2023

on starting to deploy java SDK during pre-release, but using https://github.com/pulumi/pulumi-package-publisher/tree/main to reduce duplication.

There are a couple open questions:

  • I was getting complaints about the tree not being clean when building the java version. Not sure why it is different than the other builds. For now, prerelease skips it as an example.
  • Tests are not enabled for java, since I believe it requires writing some test like for the other languages. Can take a look, but have not yet.
  • Several secrets need. At least OSSRH_USERNAME and OSSRH_PASSWORD to be able to push the Central repository (https://central.sonatype.org/publish/publish-guide/) and JAVA_SIGNING_KEY_ID/JAVA_SIGNING_KEY/JAVA_SIGNING_PASSWORD for GPG signing (IIUC). Since I obviously do not own the repo, I can not add these. It seems like the Central Repository set-up will require a few day wait.

Other notes:

  • I've delete master.yml, since it seems unused, and is one less place to copy similar logic.
  • I've updated prelease.yml and release.yml, but not much on main.yml, since it surprised me it did publishing of sdks. Open to discussion on what should be in it.

Testing:

Please let me know your thoughts. Thanks!


Ellipsis 🚀 This PR description was created by Ellipsis for commit 8ad69dd.

Summary:

This PR initiates the export of the Java SDK during pre-release, deletes an unused file, and updates two workflow files, but notes that tests for Java are not enabled and several secrets are needed.

Key points:

  • Started exporting Java SDK during pre-release using pulumi-package-publisher
  • Deleted unused master.yml file
  • Updated prerelease.yml and release.yml
  • Noted that tests are not enabled for Java
  • Noted that several secrets are needed for the Central repository and GPG signing

Generated with ❤️ by ellipsis.dev

on starting to deploy java SDK during pre-release, but using
https://github.com/pulumi/pulumi-package-publisher/tree/main to reduce
duplication.

There are a couple open questions:
* I was getting complaints about the tree not being clean when building
  the java version. Not sure why it is different than the other builds.
* Tests are not enabled for java, since I believe it requires writing
  some test like for the other languages. Can take a look, but have not
  yet.

Other notes:
* I've delete master.yml, since it seems unused, and is one less place
  to copy similar logic.
* I've updated prelease.yml and release.yml, but not much on main.yml,
  since it surprised me it did publishing of sdks. Open to discussion on
  what should be in it.
Copy link

github-actions bot commented Nov 1, 2023

PR is now waiting for a maintainer to run the acceptance tests.
Note for the maintainer: To run the acceptance tests, please comment /run-acceptance-tests on the PR

@kylepl
Copy link
Author

kylepl commented Nov 1, 2023

This is meant to address #7.

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

No problems found on commit 8ad69dd


Something look wrong? You can customize Ellipsis by editing the ellipsis.yaml for this repository.

Generated with ❤️ by ellipsis.dev

@ringods
Copy link
Member

ringods commented Oct 29, 2024

@kylepl I'm closing your PR for now. Publishing Java packages for all Pulumiverse providers is not possible for now. Here is the reason:

pulumiverse/infra#44 (comment)

@ringods ringods closed this Oct 29, 2024
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