Skip to content

Conversation

@red-hat-konflux-kflux-prd-rh03
Copy link
Contributor

@red-hat-konflux-kflux-prd-rh03 red-hat-konflux-kflux-prd-rh03 bot commented Oct 28, 2025

This PR contains the following updates:

Package Update Change
fedora major 42 -> 43

Warning

Some dependencies could not be looked up. Check the warning logs for more information.


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

To execute skipped test pipelines write comment /ok-to-test.


Documentation

Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.

Signed-off-by: red-hat-konflux-kflux-prd-rh03 <206760901+red-hat-konflux-kflux-prd-rh03[bot]@users.noreply.github.com>
@dustymabe
Copy link
Member

@aaradhak - there are some manual steps required here. Let me know if you need help.

@aaradhak
Copy link
Member

aaradhak commented Nov 6, 2025

I think @marmijo is looking into this cosa bump part.

@marmijo
Copy link
Member

marmijo commented Nov 7, 2025

there are some manual steps required here. Let me know if you need help.

I think #4366 should unblock this PR.

@dustymabe
Copy link
Member

I think the prow jobs need some updates too so we'll probably have to ignore/override those tests to merge this. Not sure if we have that documented anywhere.

See openshift/release#65351

dustymabe
dustymabe previously approved these changes Nov 7, 2025
Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@red-hat-konflux-kflux-prd-rh03
Copy link
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

dustymabe
dustymabe previously approved these changes Nov 7, 2025
Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@dustymabe
Copy link
Member

16:12:53  ************* Module cosalib.cmdlib

16:12:53  src/cosalib/cmdlib.py:412:29: E1120: No value for argument 'self' in function call (no-value-for-parameter)

16:12:53  ************* Module cmd-basearch

16:12:53  src/cmd-basearch:7:6: E1120: No value for argument 'self' in function call (no-value-for-parameter)

16:12:53  ************* Module commitmeta_to_json

16:12:53  src/commitmeta_to_json:28:4: E1120: No value for argument 'self' in unbound method call (no-value-for-parameter)

16:12:56  make: *** [Makefile:58: pycheck] Error 2

looks like newer versions of linting libraries in F43 are stricter and there's a little cleanup for us to do.

pylint is treating GObject-introspected methods (from gi.repository) as
unbound instance methods that require a self argument, even though at
runtime these calls are valid and work as expected. This causes false
positive E1120 (no-value-for-parameter) errors on calls such as
RpmOstree.get_basearch() and Json.Generator.new().

Add disable comments to the affected GI calls to keep pylint from
complaining while preserving the existing behavior.
@marmijo marmijo force-pushed the renovate/main/fedora-43.x branch from a167f45 to 86974da Compare November 14, 2025 21:34
@marmijo
Copy link
Member

marmijo commented Nov 19, 2025

/retest-required

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@marmijo marmijo merged commit 95ff89d into main Nov 19, 2025
8 of 9 checks passed
@marmijo marmijo deleted the renovate/main/fedora-43.x branch November 19, 2025 19:00
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.

4 participants