Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6.16.z] Avoid calling decode on str #16304

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #16214

Problem Statement

In tests we see this failure:

AttributeError: 'str' object has no attribute 'decode'. Did you mean: 'encode'?

Solution

This implies that the strerr value is a str, which you can't decode. I can't find the specific change, but this implies the result used to be a byte string and is now decoded elsewhere.

Related Issues

* Avoid calling decode on str

In tests we see this failure:

    AttributeError: 'str' object has no attribute 'decode'. Did you mean: 'encode'?

This implies that the strerr value is a str, which you can't decode. I
can't find the specific change, but this implies the result used to be a
byte string and is now decoded elsewhere.

* Update tests/foreman/destructive/test_rename.py

Co-authored-by: Peter Ondrejka <[email protected]>

* Update tests/foreman/destructive/test_rename.py

Co-authored-by: Evgeni Golov <[email protected]>

---------

Co-authored-by: Peter Ondrejka <[email protected]>
Co-authored-by: Jameer Pathan <[email protected]>
Co-authored-by: Evgeni Golov <[email protected]>
(cherry picked from commit 5c05d4f)
@Satellite-QE Satellite-QE requested a review from a team as a code owner September 10, 2024 11:27
@Satellite-QE Satellite-QE added 6.16.z Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Sep 10, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/destructive/test_rename.py::test_negative_rename_sat_wrong_passwd

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Sep 10, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 8544
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/destructive/test_rename.py::test_negative_rename_sat_wrong_passwd --external-logging
Test Result : ================= 1 passed, 17 warnings in 1869.80s (0:31:09) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Sep 10, 2024
@Satellite-QE Satellite-QE merged commit b1632c9 into 6.16.z Sep 10, 2024
17 of 18 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.16.z-5c05d4fd72c0ad5472c0d8cc1a3702cce27f6f9b branch September 10, 2024 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.16.z Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants