Skip to content

Publishing artifacts to JFrog#839

Merged
vmaheshw merged 5 commits intolinkedin:masterfrom
shrinandthakkar:publishing_artifacts_to_jfrog
Jun 23, 2021
Merged

Publishing artifacts to JFrog#839
vmaheshw merged 5 commits intolinkedin:masterfrom
shrinandthakkar:publishing_artifacts_to_jfrog

Conversation

@shrinandthakkar
Copy link
Copy Markdown
Collaborator

@shrinandthakkar shrinandthakkar commented Jun 22, 2021

As bintray is deprecated, adding support for pushing Brooklin artifacts to JFrog simultaneously with bintray.

Running and Testing steps:

  1. Publish artifacts to JFrog. (To be able to publish artifacts to JFrog, you need to create an account at JFrog and use your LinkedIn username and API key for pushing artifacts)
  2. ELR (External Library Request) the necessary artifacts to the internal LinkedIn DDS repo from the private Brooklin repo at https://go/elr
  3. Update the respective MP to pull the artifacts from the DDS external repo, for example (https://rb.corp.linkedin.com/r/2645764/diff/4/)
  4. run mint validate && mint build on that MP to ensure it fetches all those libraries and passes all the tests.

FAQs:

a) Command to publish artifacts
./gradlew artifactoryPublish

b) How to find the artifacts? Any webpage which has all the artifacts
All the published artifacts can be found in the JFrog Brooklin repo at https://linkedin.jfrog.io/ui/repos/tree/General/brooklin

c) How does the versioning work for these artifacts?
We usually follow Semantic Versioning which is explained here, https://github.com/linkedin/brooklin/wiki/Developer-Guide. But along with that, we have to make sure that the newer version doesn't already exist in JFrog artifactory.

Important: DO NOT REPORT SECURITY ISSUES DIRECTLY ON GITHUB.
For reporting security issues and contributing security fixes,
please, email security@linkedin.com instead, as described in
the contribution guidelines.

Please, take a minute to review the contribution guidelines at:
https://github.com/linkedin/Brooklin/blob/master/CONTRIBUTING.md

@shrinandthakkar shrinandthakkar requested a review from vmaheshw June 22, 2021 00:01
Copy link
Copy Markdown
Collaborator

@vmaheshw vmaheshw left a comment

Choose a reason for hiding this comment

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

  1. Add description to the PR.
  2. Please share your testing steps (You can use internal Jira ticket to show the testing.)
  3. Sorry, I'm not aware of this part: Share the command that needs to be used to publish an artifact. Who will have access to run the artifact creation command?

vmaheshw
vmaheshw previously approved these changes Jun 22, 2021
Copy link
Copy Markdown
Collaborator

@vmaheshw vmaheshw left a comment

Choose a reason for hiding this comment

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

Some more questions on artifact:

a) How to find the artifacts ? Any webpage which has all the artifacts
b) How does the versioning works for these artifacts?

This information will be useful for everyone.

jzakaryan
jzakaryan previously approved these changes Jun 22, 2021
Copy link
Copy Markdown
Collaborator

@jzakaryan jzakaryan left a comment

Choose a reason for hiding this comment

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

LGTM. It's worth clarifying the copyright year thing before checking this in.

@shrinandthakkar shrinandthakkar dismissed stale reviews from jzakaryan and vmaheshw via 32b30c3 June 22, 2021 22:46
jzakaryan
jzakaryan previously approved these changes Jun 22, 2021
@vmaheshw vmaheshw merged commit e3b8655 into linkedin:master Jun 23, 2021
vmaheshw pushed a commit to vmaheshw/brooklin that referenced this pull request Mar 1, 2022
As bintray is deprecated, adding support for pushing Brooklin artifacts to JFrog simultaneously with bintray.
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.

3 participants