Skip to content

fix: disable doc generation for OpenStack clients (cliff sphinxext Python 3.14)#16459

Merged
christopherco merged 1 commit intotomls/base/mainfrom
liunan/disable_doc
Apr 4, 2026
Merged

fix: disable doc generation for OpenStack clients (cliff sphinxext Python 3.14)#16459
christopherco merged 1 commit intotomls/base/mainfrom
liunan/disable_doc

Conversation

@liunan-ms
Copy link
Copy Markdown
Contributor

cliff's sphinxext uses argparse._format_actions_usage() which was removed in Python 3.14.
This PR disables doc generation via spec overlay (%global with_doc 0) for python-heatclient, python-manilaclient, python-octaviaclient, and python-openstackclient until the upstream cliff fix (openstack/cliff@391261c) is released.

Also disables %check for python-oslo-config which has a test failure also due to argparse removal in python 3.14.

Validation: Koji build

Copilot AI review requested due to automatic review settings April 3, 2026 22:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Disables documentation generation for several OpenStack client RPMs to work around a Python 3.14 incompatibility in cliff.sphinxext, and disables %check for python-oslo-config due to an argparse-related test failure.

Changes:

  • Add spec overlays to set %global with_doc 0 for python-heatclient, python-manilaclient, python-octaviaclient, and python-openstackclient.
  • Remove those components’ bare entries from components-full.toml (since they now have dedicated *.comp.toml files).
  • Add python-oslo-config to the global check-skip list.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
base/comps/python-openstackclient/python-openstackclient.comp.toml New overlay disabling doc generation (with_doc 0) with upstream reference/TODO.
base/comps/python-octaviaclient/python-octaviaclient.comp.toml New overlay disabling doc generation (with_doc 0) with upstream reference/TODO.
base/comps/python-manilaclient/python-manilaclient.comp.toml New overlay disabling doc generation (with_doc 0) with upstream reference/TODO.
base/comps/python-heatclient/python-heatclient.comp.toml New overlay disabling doc generation (with_doc 0) with upstream reference/TODO.
base/comps/components-full.toml Removes bare component entries for the newly-dedicated components.
base/comps/component-check-disablement.toml Disables %check for python-oslo-config via the check-skip component group.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…thon 3.14)

cliff's sphinxext uses argparse._format_actions_usage() which was
removed in Python 3.14. Disable doc generation via spec overlay
(%global with_doc 0) for python-heatclient, python-manilaclient,
python-octaviaclient, and python-openstackclient until the upstream
cliff fix (openstack/cliff@391261c) is released.

Also add python-oslo-config to check-skip-initial-failures.

Add ^ and $ anchors to prevent unintended substring matches on
commented lines or other macro uses. Consistent with the pattern
used in python-stestr.comp.toml.
Copy link
Copy Markdown
Collaborator

@christopherco christopherco left a comment

Choose a reason for hiding this comment

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

I squashed the changes together since this is all one logical change and so the rebase merge has the proper final git history after merge. Re-ran testing and came back green. :shipit:

@christopherco christopherco merged commit e00412a into tomls/base/main Apr 4, 2026
5 checks passed
@christopherco christopherco deleted the liunan/disable_doc branch April 4, 2026 04:38
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