Skip to content

HDDS-14749. Run update-ozone-site-config-doc after generate-config-doc#9854

Merged
sarvekshayr merged 3 commits intoapache:masterfrom
adoroszlai:HDDS-14749
Mar 2, 2026
Merged

HDDS-14749. Run update-ozone-site-config-doc after generate-config-doc#9854
sarvekshayr merged 3 commits intoapache:masterfrom
adoroszlai:HDDS-14749

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

  • update-ozone-site-config-doc currently only runs if full CI succeeds, so website update can be delayed by intermittent failures. It should run after successful generate-config-doc, since it does not depend on tests.
  • Rename the generate job in generate-config-doc to clarify its name within the CI workflow: before
  • Move some variable definitions to env from run block.

https://issues.apache.org/jira/browse/HDDS-14749

How was this patch tested?

  • Full CI in fork, but update-ozone-site-config-doc is skipped due to conditions.
  • Temporary tweaks to trigger update-ozone-site-config-doc in fork:
    • removed conditions
    • added debug output for event attributes with which the PR body is generated:
      workflow: build-branch
      run_id: 22547417135
      ref_name: HDDS-14749-test
      sha: 6685253f722cc96558a1cb127498859728c7b86f
      

@adoroszlai adoroszlai requested a review from sarvekshayr March 1, 2026 19:48
@adoroszlai adoroszlai self-assigned this Mar 1, 2026
Copy link
Contributor

@sarvekshayr sarvekshayr left a comment

Choose a reason for hiding this comment

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

Thanks for the much-needed improvements @adoroszlai. LGTM.

@@ -130,7 +123,6 @@ jobs:
cd ozone-site

# Use single branch name
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: This comment can be removed.

@adoroszlai adoroszlai requested a review from sarvekshayr March 2, 2026 07:40
@sarvekshayr sarvekshayr merged commit b19addf into apache:master Mar 2, 2026
@adoroszlai adoroszlai deleted the HDDS-14749 branch March 2, 2026 08:01
@adoroszlai
Copy link
Contributor Author

Thanks @sarvekshayr for reviewing and merging this.

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