Skip to content
This repository was archived by the owner on Feb 10, 2021. It is now read-only.

Update version to match Podman #34

Closed
wants to merge 1 commit into from
Closed

Conversation

rhatdan
Copy link
Member

@rhatdan rhatdan commented Jun 28, 2019

Signed-off-by: Daniel J Walsh [email protected]

Signed-off-by: Daniel J Walsh <[email protected]>
@rhatdan
Copy link
Member Author

rhatdan commented Jun 28, 2019

@jwhonce @4383 @baude @saschagrunert PTAL

@rhatdan
Copy link
Member Author

rhatdan commented Jun 28, 2019

Want to add a version to match podman and then cut a tag/release.

@rhatdan
Copy link
Member Author

rhatdan commented Jun 28, 2019

@toabctl PTAL

@@ -16,7 +16,7 @@

setup(
name='podman',
version=os.environ.get('PODMAN_VERSION', '0.0.0'),
version=os.environ.get('PODMAN_VERSION', '1.4.3'),
Copy link
Collaborator

@4383 4383 Jun 28, 2019

Choose a reason for hiding this comment

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

It's not an issue to merge these changes but they will be removed if we merge my changes.

Also I've some comments:

  1. My proposed changes on Improve packaging by using PBR #25 would avoid to do this and would manage this by using git tag. With my changes you just need to publish a new git tag on github who refer your version of podman and it will be automatically published on pypi without any manual handling, and you don't need to take care about the version management, just publish a git tag...

  2. I don't know in which cases the PODMAN_VERSION env var can exist on the system, I suppose when podman is installed, so in this case if we install a different version of python-podman not the really the same than podman you need to assume that they can be incompatible or things like that... I'm sure it's a good things to retrieve the python client version from the podman version...
    Also how to manage the client version if we want to install from pypi? If you choose to use the env var to determine the python package version, and if you build it locally and the then publish it on pypi you need to take care to have the right podman installed version.

I hope my comment can help you.

Do not hesitate to ask things if needed.

Copy link
Member Author

Choose a reason for hiding this comment

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

I am willing to entertain other ideas. The version of Podman that we build with python-podman should be pretty close. I would expect that podman-python 1.4.3 should work with podman 1.4.3 or later versions. Might work with older versions.

@rhatdan
Copy link
Member Author

rhatdan commented Jun 28, 2019

Closing since I took @4383 Version.

@rhatdan rhatdan closed this Jun 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants