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

Publishing jars to org.opensearch.opensearch-job-scheduler #336

Merged
merged 1 commit into from
Feb 24, 2023
Merged

Publishing jars to org.opensearch.opensearch-job-scheduler #336

merged 1 commit into from
Feb 24, 2023

Conversation

joshpalis
Copy link
Member

Description

Updates org.opensearch.opensearch-job-scheduler with maven jars

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Joshua Palis <jpalis@amazon.com>
@joshpalis joshpalis requested review from dbwiddis, gaiksaya and a team February 24, 2023 00:04
@joshpalis
Copy link
Member Author

tree snapshots
snapshots
└── org
    └── opensearch
        ├── opensearch-job-scheduler
        │   ├── 3.0.0.0-SNAPSHOT
        │   │   ├── maven-metadata.xml
        │   │   ├── maven-metadata.xml.md5
        │   │   ├── maven-metadata.xml.sha1
        │   │   ├── maven-metadata.xml.sha256
        │   │   ├── maven-metadata.xml.sha512
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.jar
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.jar.md5
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.jar.sha1
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.jar.sha256
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.jar.sha512
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-javadoc.jar
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-javadoc.jar.md5
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-javadoc.jar.sha1
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-javadoc.jar.sha256
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-javadoc.jar.sha512
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.module
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.module.md5
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.module.sha1
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.module.sha256
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.module.sha512
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.md5
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.sha1
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.sha256
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.sha512
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-sources.jar
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-sources.jar.md5
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-sources.jar.sha1
        │   │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-sources.jar.sha256
        │   │   └── opensearch-job-scheduler-3.0.0.0-20230224.000723-1-sources.jar.sha512
        │   ├── maven-metadata.xml
        │   ├── maven-metadata.xml.md5
        │   ├── maven-metadata.xml.sha1
        │   ├── maven-metadata.xml.sha256
        │   └── maven-metadata.xml.sha512
        ├── opensearch-job-scheduler-spi
        │   ├── 3.0.0.0-SNAPSHOT
        │   │   ├── maven-metadata.xml
        │   │   ├── maven-metadata.xml.md5
        │   │   ├── maven-metadata.xml.sha1
        │   │   ├── maven-metadata.xml.sha256
        │   │   ├── maven-metadata.xml.sha512
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.jar
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.jar.md5
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.jar.sha1
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.jar.sha256
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.jar.sha512
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-javadoc.jar
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-javadoc.jar.md5
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-javadoc.jar.sha1
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-javadoc.jar.sha256
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-javadoc.jar.sha512
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.pom
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.pom.md5
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.pom.sha1
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.pom.sha256
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1.pom.sha512
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-sources.jar
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-sources.jar.md5
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-sources.jar.sha1
        │   │   ├── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-sources.jar.sha256
        │   │   └── opensearch-job-scheduler-spi-3.0.0.0-20230224.000723-1-sources.jar.sha512
        │   ├── maven-metadata.xml
        │   ├── maven-metadata.xml.md5
        │   ├── maven-metadata.xml.sha1
        │   ├── maven-metadata.xml.sha256
        │   └── maven-metadata.xml.sha512
        └── plugin
            └── opensearch-job-scheduler
                ├── 3.0.0.0-SNAPSHOT
                │   ├── maven-metadata.xml
                │   ├── maven-metadata.xml.md5
                │   ├── maven-metadata.xml.sha1
                │   ├── maven-metadata.xml.sha256
                │   ├── maven-metadata.xml.sha512
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.md5
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.sha1
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.sha256
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.pom.sha512
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.zip
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.zip.md5
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.zip.sha1
                │   ├── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.zip.sha256
                │   └── opensearch-job-scheduler-3.0.0.0-20230224.000723-1.zip.sha512
                ├── maven-metadata.xml
                ├── maven-metadata.xml.md5
                ├── maven-metadata.xml.sha1
                ├── maven-metadata.xml.sha256
                └── maven-metadata.xml.sha512

@gaiksaya
Copy link
Member

We are not publishing javadoc and sources jar https://aws.oss.sonatype.org/content/repositories/snapshots/org/opensearch/opensearch-job-scheduler/3.0.0.0-SNAPSHOT/
@prudhvigodithi is that okay to publish here? Or do we need to modify the task to exclude them?

@codecov-commenter
Copy link

Codecov Report

Merging #336 (e14d701) into main (64d8f5e) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff            @@
##               main     #336   +/-   ##
=========================================
  Coverage     30.23%   30.23%           
  Complexity       97       97           
=========================================
  Files            22       22           
  Lines          1128     1128           
  Branches        107      107           
=========================================
  Hits            341      341           
  Misses          766      766           
  Partials         21       21           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@prudhvigodithi
Copy link
Member

Hey Its ok to publish the javadoc and source jar since the source code is already open sourced, thats the default behaviour with publish as well, I see example common-utils already doing it, so the PR LGTM!
@gaiksaya @joshpalis @dblock @bbarani

@dbwiddis dbwiddis merged commit 0711f3b into opensearch-project:main Feb 24, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 24, 2023
joshpalis added a commit that referenced this pull request Feb 27, 2023
(cherry picked from commit 0711f3b)
Signed-off-by: Joshua Palis <jpalis@amazon.com>
dbwiddis pushed a commit that referenced this pull request Feb 27, 2023
(cherry picked from commit 0711f3b)
Signed-off-by: Joshua Palis <jpalis@amazon.com>

Co-authored-by: Joshua Palis <jpalis@amazon.com>
prudhvigodithi pushed a commit to prudhvigodithi/job-scheduler that referenced this pull request May 9, 2023
…h-project#336) (opensearch-project#337)

(cherry picked from commit 0711f3b)
Signed-off-by: Joshua Palis <jpalis@amazon.com>

Co-authored-by: Joshua Palis <jpalis@amazon.com>
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
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.

6 participants