Skip to content

Conversation

@aujjwal-redhat
Copy link
Collaborator

Package created:
Name: odinControl

Sub-directory: odinControl
Fix: #11
Signed-off-by: aujjwal-redhat [email protected]

Package created:
Name: odinControl

Sub-directory: odinControl

Signed-off-by: aujjwal-redhat <[email protected]>
@aujjwal-redhat aujjwal-redhat added the enhancement New feature or request label Feb 22, 2021
rexe was defined in the outer module.
Added sys to use the Rexe class.

Signed-off-by: aujjwal-redhat <[email protected]>
sample => package

docs=> docs for odinControl
example=> the example folder
tests => test cases

Signed-off-by: aujjwal-redhat <[email protected]>
Added the commands that can help in running gluster from Source Code

Signed-off-by: aujjwal-redhat <[email protected]>
author='Srijan Sivakumar',
author_email='[email protected]',
license=license,
packages=['sample'],
Copy link
Owner

Choose a reason for hiding this comment

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

I don't think the package name should be sample..

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

packages tells the place where the package rexe is. In the refactored code I had put it in sample. Thats why I had added the refactored part to this PR only
Will close this PR then.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

By doing pip install . it was working on my system.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Screenshot from 2021-02-23 10-38-38

I guess sample just says that you will find an init.py in this package.

Copy link
Owner

Choose a reason for hiding this comment

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

instead of doing the pip install . I'd say check the document as to how to use the test pypi environment and upload the package using twine and then install it in a virtualenv and see if the functionality works. Basically the package will be imported for it's class Rexe. So we need to check if that works.

Copy link
Owner

Choose a reason for hiding this comment

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

Also, on checking with the twine to see if the distribution is syntactically correct, I get errors for the operation wherein we read the license.

Copy link
Owner

Choose a reason for hiding this comment

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

instead of specifying the package, we can use the find_package() function provided by the setuptools module.

@@ -0,0 +1 @@
import rexe
Copy link
Owner

Choose a reason for hiding this comment

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

did you try pushing the package to testpypi and try installing it from the created example pip package.. you can check the documentation : https://packaging.python.org/tutorials/packaging-projects/.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ok will try it

@@ -0,0 +1,40 @@
name: Command Execution
Copy link
Owner

Choose a reason for hiding this comment

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

Please address this as a separate PR as this addresses a different issue

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok

Will generate a new PR for this separately.

Signed-off-by: aujjwal-redhat <[email protected]>
author='Srijan Sivakumar',
author_email='[email protected]',
license=license,
packages=['sample'],
Copy link
Owner

Choose a reason for hiding this comment

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

instead of specifying the package, we can use the find_package() function provided by the setuptools module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package creation

2 participants