Bug 1428350

Summary: [CLI] pcs currently does not support "pcs config (export ... | import-cman ...)" because of Python run-time mismatch with clufter (3 vs. 2)
Product: [Fedora] Fedora Reporter: Jan Pokorný [poki] <jpokorny>
Component: pcsAssignee: Tomas Jelinek <tojeline>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: anprice, cfeist, idevat, jpokorny, tojeline
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: pcs-0.9.156-2.fc24 pcs-0.9.156-2.fc25 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-04-03 22:19:35 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1428382    
Bug Blocks:    
Attachments:
Description Flags
proposed fix none

Description Jan Pokorný [poki] 2017-03-02 12:05:22 UTC
This bug is a placeholder for any action items regarding integration
that may emerge as soon as clufter supports Python 3.

No action required for now.

To document current state with python-clufter indeed installed:
$ pcs config export pcs-commands
> Error: Unable to perform export due to missing python-clufter package

Comment 1 Jan Pokorný [poki] 2017-03-21 12:35:15 UTC
There's one change in the spec needed for sure:
-Requires: python-clufter
+Requires: python3-clufter

This is applicable wherever "pcs" executable relies on Python 3,
so this covers all currently supported Fedora releases.

Moreover, there will likely be a small change required so as to
drop "encode('utf-8')" for the products of clufter library because
"bytestring" protocol already implies a mere sequence of bytes without
higher-level string interpretation (as relevant in Python 3 context).

Comment 2 Jan Pokorný [poki] 2017-03-22 16:25:23 UTC
re [comment 1]:

> Moreover, there will likely be a small change required so as to
> drop "encode('utf-8')" for the products of clufter library because
> "bytestring" protocol already implies a mere sequence of bytes without
> higher-level string interpretation (as relevant in Python 3 context).

That's indeed the case:
https://github.com/ClusterLabs/pcs/pull/130

Tested with: pcs config {import-cman,export pcs-commands*}

Comment 3 Tomas Jelinek 2017-03-23 11:41:09 UTC
Created attachment 1265690 [details]
proposed fix

Comment 4 Fedora Update System 2017-03-23 13:03:01 UTC
pcs-0.9.156-2.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-71e69a691b

Comment 5 Fedora Update System 2017-03-23 13:03:18 UTC
pcs-0.9.156-2.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-7accc8010b

Comment 6 Fedora Update System 2017-03-24 19:52:31 UTC
pcs-0.9.156-2.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-7accc8010b

Comment 7 Fedora Update System 2017-03-24 20:23:33 UTC
pcs-0.9.156-2.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-71e69a691b

Comment 8 Fedora Update System 2017-04-03 22:19:35 UTC
pcs-0.9.156-2.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.

Comment 9 Fedora Update System 2017-04-03 22:53:17 UTC
pcs-0.9.156-2.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.