Project

General

Profile

Actions

Bug #16278

closed

zypper_pattern module does not work in python3

Added by Félix DALLIDET over 4 years ago. Updated almost 2 years ago.

Status:
Released
Priority:
N/A
Category:
Generic methods
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

Still true, we are opening the output from some commands as "text" and not "bytes" since we are using the universal_newlines type in a function.


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #15522: zypper_pattern module does not work in python3ReleasedAlexis MoussetActions
Actions #1

Updated by Félix DALLIDET over 4 years ago

  • Related to Bug #15522: zypper_pattern module does not work in python3 added
Actions #2

Updated by Félix DALLIDET over 4 years ago

  • Status changed from New to In progress
  • Assignee set to Félix DALLIDET
Actions #3

Updated by Félix DALLIDET over 4 years ago

This only affect the zypper_pattern module, mostly because it is the only module which call cmd based on other cmd outputs.
Since python3, bytes and str are separated and we must pass on every string manipulation previously done and update them to be sure they never mix bytes and str.

Bytes are mostly coming from cmd outputs, but in some cases, the cmd output outputs str, I think the "universal_newline" option it the faulty member here.

Actions #4

Updated by Alexis Mousset over 4 years ago

  • Target version changed from 6.0.0 to 6.0.1
Actions #5

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 6.0.1 to 6.0.2
Actions #6

Updated by Félix DALLIDET over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Félix DALLIDET to Benoît PECCATTE
  • Pull Request set to https://github.com/Normation/ncf/pull/1132
Actions #7

Updated by Félix DALLIDET over 4 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:40c69131ef48cc38570e5675b2a8bca1c40f21c7.

Actions #8

Updated by Félix DALLIDET over 4 years ago

  • Fix check changed from To do to Checked
Actions #9

Updated by Alexis Mousset over 4 years ago

  • Subject changed from zypper_pattern module does not work in python3 Bis to zypper_pattern module does not work in python3
  • Name check changed from To do to Reviewed
Actions #10

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.0.2 which was released today.

Actions #11

Updated by Alexis Mousset almost 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Generic methods - Package Management to Generic methods
Actions

Also available in: Atom PDF