Skip to content

Conversation

@apoorvmittal10
Copy link
Contributor

Shadow jar plugin version 8.1.1 causes issue when shadowed artifacts are signed and published. The issue occurs because archiveBaseName and archiveClassifier are not honoured correctly by publication and tries to find the shadow jar which doesn't exist - see GradleUp/shadow#901

This PR reverts the shadow plugin version upgrade added in the PR: #13625 and also reverts the workaround to publish signed shadowed jar in the PR: #15127

  • I have verified the resultant pom for kafka-clients which does include the runtime dependencies.
  • Verified the kafka-clients.jar has correct shaded classes
  • Verified the release pipeline i.e. signing and publishing works correctly to remote maven repository.

@ijuma The query asked in the PR: #15127 still stands as Gradle Module Metadata is never generated with shadow jar plugin.

cc: @stanislavkozlovski @AndrewJSchofield @xvrl

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

Copy link
Member

@AndrewJSchofield AndrewJSchofield left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@mimaison mimaison left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@mimaison mimaison merged commit b35c62e into apache:trunk Feb 2, 2024
@mimaison
Copy link
Member

mimaison commented Feb 2, 2024

@stanislavkozlovski I let you backport to 3.7

@ijuma
Copy link
Member

ijuma commented Feb 2, 2024

@apoorvmittal10 Thanks for getting to the bottom of this!

stanislavkozlovski pushed a commit that referenced this pull request Feb 5, 2024
@stanislavkozlovski
Copy link
Contributor

cherry-picked to 3.7

yyu1993 pushed a commit to yyu1993/kafka that referenced this pull request Feb 15, 2024
@iit2009060
Copy link
Contributor

iit2009060 commented Feb 25, 2024

@apoorvmittal10 I am facing an error after this changes I pulled into local

- Gradle Core Plugins (plugin is not in 'org.gradle' namespace)
- Plugin Repositories (could not resolve plugin artifact
com.github.johnrengelman.shadow:com.github.johnrengelman.shadow.gradle.plugin:8.1.0')
  Searched in the following repositories:
    Gradle Central Plugin Repository

Can you help ?

@apoorvmittal10
Copy link
Contributor Author

@apoorvmittal10 I am facing an error after this changes I pulled into local

- Gradle Core Plugins (plugin is not in 'org.gradle' namespace)
- Plugin Repositories (could not resolve plugin artifact
com.github.johnrengelman.shadow:com.github.johnrengelman.shadow.gradle.plugin:8.1.0')
  Searched in the following repositories:
    Gradle Central Plugin Repository

Can you help ?

Hi Arpit, the plugin is configured correctly and the version is available. I would advise to check if there is anything custom configured in you setup which point to different repository.

clolov pushed a commit to clolov/kafka that referenced this pull request Apr 5, 2024
Phuc-Hong-Tran pushed a commit to Phuc-Hong-Tran/kafka that referenced this pull request Jun 6, 2024
dajac pushed a commit that referenced this pull request Feb 22, 2025
…16295)" (#19000)

The PR: #16295 which changed the
shadow plugin version has re-introduced the issue with publication of
signed jars, fixed in: #15308.

This PR reverts #16295 to unblock
Kafka 4.0 release. And we should aim for fixing the issue permanently as
being discussed here: #18018.

Seems same issue was encountered during `Kafka-3.9` release and same
commit was reverted in 3.9 as well:
#17218.

Reviewers: David Jacot <[email protected]>
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.

6 participants