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

Changes for Python 3 compatibility (CFME 5.11/RHEL8) #557

Merged

Conversation

pemcg
Copy link

@pemcg pemcg commented Aug 7, 2019

These seem to be the bare minimum changes needed to get the manageiq-automate role working in CFME 5.11 on RHEL8 ( Python3).

https://docs.python.org/3.0/whatsnew/3.0.html describes the changes, but suggests that some constructs may be better re-worked to be optimised for Python 3. I haven't done that.

https://bugzilla.redhat.com/show_bug.cgi?id=1738542

@gmcculloug gmcculloug requested a review from syncrou August 7, 2019 12:12
@gmcculloug gmcculloug self-assigned this Aug 7, 2019
@miq-bot
Copy link
Member

miq-bot commented Aug 7, 2019

Checked commit pemcg@e302d69 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@gmcculloug
Copy link
Member

Some overlap with #556.

@pemcg
Copy link
Author

pemcg commented Aug 7, 2019

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3408

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 96.936%

Totals Coverage Status
Change from base Build 3392: 0.005%
Covered Lines: 2911
Relevant Lines: 3003

💛 - Coveralls

Copy link
Contributor

@syncrou syncrou left a comment

Choose a reason for hiding this comment

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

LGTM 👍 I was able to verify these changes work on Python 2 as well.

@gmcculloug gmcculloug merged commit 805a31b into ManageIQ:master Aug 9, 2019
simaishi pushed a commit that referenced this pull request Aug 12, 2019
…ility

Changes for Python 3 compatibility (CFME 5.11/RHEL8)

(cherry picked from commit 805a31b)

https://bugzilla.redhat.com/show_bug.cgi?id=1738542
@simaishi
Copy link
Contributor

Ivanchuk backport details:

$ git log -1
commit 594756ec4207a9101978d049f22af0a6de3d27aa
Author: Greg McCullough <[email protected]>
Date:   Fri Aug 9 18:04:04 2019 -0400

    Merge pull request #557 from pemcg/manageiq-automate_python3_compatibility
    
    Changes for Python 3 compatibility (CFME 5.11/RHEL8)
    
    (cherry picked from commit 805a31b1fcc8abef623e31fffd5670c193f2c763)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1738542

@gmcculloug gmcculloug added this to the Sprint 118 Ending Aug 19, 2019 milestone Aug 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants