Skip to content

Conversation

@floatingstatic
Copy link
Collaborator

@floatingstatic floatingstatic commented Nov 30, 2020

This closes #66 which I ran into installing the 0.4.1 version of this napalm pack in a fresh stackstorm v3.3.0 installation.

It appears as though the check_consistency_chatops workflow was added in #17 with nothing referencing it (no metadata file). Oddly it contains the same workflow tasks as configuration_change_workflow so I am opting to just remove the file.

@floatingstatic
Copy link
Collaborator Author

There appears to be some issue with the ci test environment related to the version of the cryptography module as part of st2common==3.4.dev0:

Installed /home/circleci/virtualenv/lib/python3.6/site-packages/eventlet-0.25.1-py3.6.egg
error: cryptography 2.8 is installed but cryptography>=3.2 is required by {'pyOpenSSL'}
make: *** [Makefile:273: .install-runners] Error 1
Original script exit code: 2

Exited with code exit status 2
CircleCI received exit code 2

I'm not sure how to proceed with fixing this.

@arm4b arm4b added the feature label Nov 30, 2020
@arm4b
Copy link
Member

arm4b commented Nov 30, 2020

Yeah, we're updating cryptography dependency in st2 core. Once the StackStorm/st2#5095 is merged, this PR CI should be ✔️ .

@arm4b
Copy link
Member

arm4b commented Nov 30, 2020

@floatingstatic Thanks for the PR!
Are you folks heavily relying on this pack in production?
If you're interested, we could give your team maintainer access to this pack in Exchange.

@floatingstatic
Copy link
Collaborator Author

@floatingstatic Thanks for the PR!
Are you folks heavily relying on this pack in production?
If you're interested, we could give your team maintainer access to this pack in Exchange.

I use st2 internally at work but all of the packs are custom (we do not use NAPALM). I do however use this pack at home for personal use. If you are looking for some help maintaining it I would be happy to assist if there is a need here. Let me know.

@arm4b
Copy link
Member

arm4b commented Dec 2, 2020

@floatingstatic Thanks! Created #68 for that.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution 👍

@arm4b arm4b merged commit 63a1ffe into StackStorm-Exchange:master Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NAPALM workflows need converting from mistral to Orquesta

2 participants