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

DSANSIBLE-98 Migrate to pytest #25

Merged
merged 4 commits into from Jul 20, 2022
Merged

DSANSIBLE-98 Migrate to pytest #25

merged 4 commits into from Jul 20, 2022

Conversation

NjM3MjY5NzAgNzA3MzA3
Copy link
Member

Convert to pytest as nose is unmaintained and doesn't work with >python3.9
Add python3.9 and 3.10 to travis and tox
Remove duplicate run with cover and py37
Move cover,docs,flake8 to python3.9

@NjM3MjY5NzAgNzA3MzA3 NjM3MjY5NzAgNzA3MzA3 added the enhancement New feature or request label Jul 20, 2022
nose is unmaintained and doesn't work with >python3.9
Convert to pytest
Add python3.9 and 3.10

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
cover and py3x are the same, no need to duplicate

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
@blearandy blearandy merged commit 34d1a49 into develop Jul 20, 2022
@blearandy blearandy deleted the feature/98/pytest branch July 20, 2022 20:09
blearandy added a commit that referenced this pull request Sep 29, 2022
* Feature/dsansible 76/setup read the docs (#17)

* Update the README to point to read the docs, and add in other readme items likes additional badges

* Feature/dsansible 76/setup read the docs (#19)

* Update the README to point to read the docs, and add in other readme items likes additional badges

* A few additional README.md changes

* Upating per Justin's comments

* Add the read the docs theme to the pyds8k documentation.

* Trying to get read the docs theme to work.

* removing import...

* code to handle theme only when on readthedocs

* code to handle theme only when on readthedocs

* code to handle theme only when on readthedocs

* add two function to pyds8k (#12)

* Revert "add two function to pyds8k (#12)" (#22)

This reverts commit eb97922.

* To incorporate the PR - #12 (#24)

(task/CSI-2929 add two function to pyds8k), by manually adding following function only:
get_host_ports_by_host

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-78 (#21)

* DSANSIBLE-78
to Support Win
1) change hard coded path separator to os.sep

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-78
to Support Win
1) change hard coded path separator to os.sep

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-28, supporting creating with id & ids (#20)

* DSANSIBLE-28, supporting creating with id & ids

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-28, supporting creating with id & ids(reformat, flake8 compliance)

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-98 Migrate to pytest (#25)

* DSANSIBLE-98 Migrate to pytest
nose is unmaintained and doesn't work with >python3.9
Convert to pytest
Add python3.9 and 3.10

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* Add python 3.9 and 3.10 to travis

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-98 Relax pytest version for 3.6

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-98 Run cover,docs,flake8 on 3.9
cover and py3x are the same, no need to duplicate

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-78 Add windows to CI (#26)

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* added create_alias_volumes call in RootVolumeMixin for alias support (#27)

* added create_alias_volumes call in RootVolumeMixin for alias support

Signed-off-by: Randy Blea <blead@us.ibm.com>

* fix line too long error

Signed-off-by: Randy Blea <blead@us.ibm.com>

* DSANSIBLE-83 Add unit test for creating alias vols

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

Co-authored-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-87 Add resource groups. (#30)

* DSANSIBLE-87 Add resource groups.

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-87 fix incorrect variable name.

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-102: add basevolume parser on volume alias request (#31)

Signed-off-by: shipeng@us.ibm.com

Signed-off-by: shipeng@us.ibm.com
Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-100: flash copy information was not correctly handled due to evolving flash copy rest api. (#32)

instead of *_flashcopy, *_cs_flashcopy should be used for flashcopy operations

for backward compatibility, move to use volume_id for pre-R8, which is deprecated.
if volume_id is empty and the calling party is a volume, volume_id is populated.

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
Signed-off-by: shipeng@us.ibm.com
Co-authored-by: Matan Carmeli <45543087+matancarmeli7@users.noreply.github.com>
Co-authored-by: shipeng <AnboAnbo@users.noreply.github.com>
Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>
Co-authored-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
blearandy added a commit that referenced this pull request Sep 29, 2022
* Feature/dsansible 76/setup read the docs (#17)

* Update the README to point to read the docs, and add in other readme items likes additional badges

* Feature/dsansible 76/setup read the docs (#19)

* Update the README to point to read the docs, and add in other readme items likes additional badges

* A few additional README.md changes

* Upating per Justin's comments

* Add the read the docs theme to the pyds8k documentation.

* Trying to get read the docs theme to work.

* removing import...

* code to handle theme only when on readthedocs

* code to handle theme only when on readthedocs

* code to handle theme only when on readthedocs

* add two function to pyds8k (#12)

* Revert "add two function to pyds8k (#12)" (#22)

This reverts commit eb97922.

* To incorporate the PR - #12 (#24)

(task/CSI-2929 add two function to pyds8k), by manually adding following function only:
get_host_ports_by_host

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-78 (#21)

* DSANSIBLE-78
to Support Win
1) change hard coded path separator to os.sep

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-78
to Support Win
1) change hard coded path separator to os.sep

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-28, supporting creating with id & ids (#20)

* DSANSIBLE-28, supporting creating with id & ids

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-28, supporting creating with id & ids(reformat, flake8 compliance)

Signed-off-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-98 Migrate to pytest (#25)

* DSANSIBLE-98 Migrate to pytest
nose is unmaintained and doesn't work with >python3.9
Convert to pytest
Add python3.9 and 3.10

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* Add python 3.9 and 3.10 to travis

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-98 Relax pytest version for 3.6

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-98 Run cover,docs,flake8 on 3.9
cover and py3x are the same, no need to duplicate

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-78 Add windows to CI (#26)

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* added create_alias_volumes call in RootVolumeMixin for alias support (#27)

* added create_alias_volumes call in RootVolumeMixin for alias support

Signed-off-by: Randy Blea <blead@us.ibm.com>

* fix line too long error

Signed-off-by: Randy Blea <blead@us.ibm.com>

* DSANSIBLE-83 Add unit test for creating alias vols

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

Co-authored-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-87 Add resource groups. (#30)

* DSANSIBLE-87 Add resource groups.

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-87 fix incorrect variable name.

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>

* DSANSIBLE-102: add basevolume parser on volume alias request (#31)

Signed-off-by: shipeng@us.ibm.com

Signed-off-by: shipeng@us.ibm.com
Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* DSANSIBLE-100: flash copy information was not correctly handled due to evolving flash copy rest api. (#32)

instead of *_flashcopy, *_cs_flashcopy should be used for flashcopy operations

for backward compatibility, move to use volume_id for pre-R8, which is deprecated.
if volume_id is empty and the calling party is a volume, volume_id is populated.

Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>

* Update __init__.py (#34)

Update release number for pyds8k

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
Signed-off-by: shipeng@us.ibm.com
Co-authored-by: Matan Carmeli <45543087+matancarmeli7@users.noreply.github.com>
Co-authored-by: shipeng <AnboAnbo@users.noreply.github.com>
Co-authored-by: shipeng@us.ibm.com <shipeng@us.ibm.com>
Co-authored-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
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.

None yet

2 participants