Skip to content

Conversation

@thrix
Copy link

@thrix thrix commented Apr 22, 2025

No description provided.

Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
thrix added a commit to teemtee/tmt that referenced this pull request Apr 22, 2025
Seems `system-upgrade` does not work anymore, use the advised
`dnf5 offline reboot` to perform the upgrade from Fedora 42.

See teemtee/upgrade#15

Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
thrix added a commit to teemtee/tmt that referenced this pull request Apr 22, 2025
Seems `system-upgrade` does not work anymore, use the advised
`dnf5 offline reboot` to perform the upgrade from Fedora 42.

See teemtee/upgrade#15

Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
Copy link
Contributor

@psss psss left a comment

Choose a reason for hiding this comment

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

Looks good.

@psss
Copy link
Contributor

psss commented Apr 22, 2025

@mmacura311, could you please have a look as well? Is this now the recommended way how to upgrade?

@psss psss requested a review from mmacura311 April 22, 2025 20:37
@mmacura311
Copy link
Collaborator

Hi @psss
according the documentation
https://docs.fedoraproject.org/en-US/quick-docs/upgrading-fedora-offline/
the user should run dnf system-upgrade download ... and afterwards dnf system-upgrade reboot so TBH I do not see a need to run the dnf5 offline which looks like lower-level functionality to perform offlne transactions.

@thrix
Copy link
Author

thrix commented Apr 23, 2025

Hi @psss according the documentation https://doI cs.fedoraproject.org/en-US/quick-docs/upgrading-fedora-offline/ the user should run dnf system-upgrade download ... and afterwards dnf system-upgrade reboot so TBH I do not see a need to run the dnf5 offline which looks like lower-level functionality to perform offlne transactions.

@mmacura311 yes, I conclude with you, unfortunately the dnf system-upgrade reboot for Fedora 41 to 42 upgrade does not actually trigger the update. Could be a bug, we do not know yet. Let's keep this open so we can eventually find out and move on unblocking tmt via this draft @psss .

@psss
Copy link
Contributor

psss commented Apr 23, 2025

Let's keep this open so we can eventually find out and move on unblocking tmt via this draft @psss .

Ack.

psss pushed a commit to teemtee/tmt that referenced this pull request Apr 23, 2025
Seems `system-upgrade` does not work anymore, use the advised
`dnf5 offline reboot` to perform the upgrade from Fedora 42.

See teemtee/upgrade#15

Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
psss added a commit to teemtee/tmt that referenced this pull request Apr 23, 2025
The `virt-customize` for adding additional deps to the tested VM
image started to fail recently. Drop it for now until we find out
what is causing it to bring the test back to stable state.

Seems `system-upgrade` does not work anymore, use the advised
`dnf5 offline reboot` to perform the upgrade from Fedora 42.
See teemtee/upgrade#15

Add `beakerlib` framework for /tests/libraries/apache test
This is required if beakerlib is not bundled into the image.
Was dropped when workarounding tests for 1.47 release, see:
#3687

Co-authored-by: Miroslav Vadkerti <mvadkert@redhat.com>
@psss
Copy link
Contributor

psss commented Apr 23, 2025

See also the following pull request where packit targets were updated:

bajertom pushed a commit to teemtee/tmt that referenced this pull request Aug 11, 2025
The `virt-customize` for adding additional deps to the tested VM
image started to fail recently. Drop it for now until we find out
what is causing it to bring the test back to stable state.

Seems `system-upgrade` does not work anymore, use the advised
`dnf5 offline reboot` to perform the upgrade from Fedora 42.
See teemtee/upgrade#15

Add `beakerlib` framework for /tests/libraries/apache test
This is required if beakerlib is not bundled into the image.
Was dropped when workarounding tests for 1.47 release, see:
#3687

Co-authored-by: Miroslav Vadkerti <mvadkert@redhat.com>
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