Skip to content

Conversation

jovial
Copy link
Contributor

@jovial jovial commented Sep 5, 2025

This sets the collection wide python interpretter. This can also be set
on a per role basis with:

  • os_openstackclient_virtualenv_python
  • os_openstacksdk_virtualenv_python

If python version changes, it will create the virtualenv with the new interpreter.

Also adds:

  • os_openstackclient_virtualenv_command

    Depending on the version of upper constraints, you may need to use a
    newer python interpreter.

@jovial
Copy link
Contributor Author

jovial commented Sep 5, 2025

Would be nice if this recreated virtualenv if python version was changing

@jovial jovial force-pushed the feature/os_openstackclient_virtualenv_command branch from 008e006 to 9ec3c13 Compare September 5, 2025 13:40
@jovial
Copy link
Contributor Author

jovial commented Sep 5, 2025

Would be nice if this recreated virtualenv if python version was changing

Done

@jovial jovial force-pushed the feature/os_openstackclient_virtualenv_command branch from 9ec3c13 to 3602a83 Compare September 5, 2025 13:42
@jovial jovial changed the title Adds os_openstackclient_virtualenv_command Adds os_virtualenv_python Sep 5, 2025
@jovial jovial force-pushed the feature/os_openstackclient_virtualenv_command branch from 3602a83 to 2adac48 Compare September 5, 2025 13:50
This sets the collection wide python interpretter. This can also be set
on a per role basis with:

- os_openstackclient_virtualenv_python
- os_openstacksdk_virtualenv_python

Also adds:

- os_openstackclient_virtualenv_command

Depending on the version of upper constraints, you may need to use a
newer python interpreter.
@jovial jovial force-pushed the feature/os_openstackclient_virtualenv_command branch from 2adac48 to baa6112 Compare September 5, 2025 13:57
@jovial jovial marked this pull request as ready for review September 5, 2025 13:58
@jovial jovial requested a review from a team as a code owner September 5, 2025 13:58
Copy link
Member

@dougszumski dougszumski left a comment

Choose a reason for hiding this comment

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

Looks fine I think

@jovial jovial merged commit 97a897b into main Sep 5, 2025
4 checks passed
@jovial jovial deleted the feature/os_openstackclient_virtualenv_command branch September 5, 2025 14:51
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.

2 participants