Skip to content

NUTCH-3204 Trigger single-node Hadoop smoke test from PR /smoke-test comments and release tags - #952

Merged
lewismc merged 1 commit into
apache:masterfrom
lewismc:NUTCH-3204
Aug 16, 2026
Merged

NUTCH-3204 Trigger single-node Hadoop smoke test from PR /smoke-test comments and release tags#952
lewismc merged 1 commit into
apache:masterfrom
lewismc:NUTCH-3204

Conversation

@lewismc

@lewismc lewismc commented Aug 15, 2026

Copy link
Copy Markdown
Member

PR for NUTCH-3204

  • I added the GitHub Actions workflow to queue the ASF Jenkins single-node Hadoop smoke job from /smoke-test (will only execute for GitHub users with write access to the Nutch repository) and release-X.Y[.Z] tag pushes
  • Updated the smoke Jenkinsfile to post/update a single PR status comment (initially a ACK which will then be updated by Jenkins with the final result)
  • Skip the Actions job unless the comment body contains /smoke-test (or the event is a tag push)

Committer-facing documentation exists on the project wiki

@lewismc lewismc self-assigned this Aug 15, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
D Security Rating on New Code (required ≥ A)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@lewismc

lewismc commented Aug 15, 2026

Copy link
Copy Markdown
Member Author

Looks like SonarCloud scanner is flagging issues with "new code" not contained in the pull request scope. I'm not sure entirely why.
Additionally, this is another feature that can only be validated once merged into master.

@lewismc lewismc changed the title Trigger single-node Hadoop smoke test from PR /smoke-test comments and release tags NUTCH-3204 Trigger single-node Hadoop smoke test from PR /smoke-test comments and release tags Aug 15, 2026

@sebastian-nagel sebastian-nagel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.

can only be validated once merged

Yes. That might result in a forth and back until smoke tests are run and also succeed.

@lewismc
lewismc merged commit 21a17b3 into apache:master Aug 16, 2026
16 checks passed
@lewismc
lewismc deleted the NUTCH-3204 branch August 16, 2026 19:47
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