Missing greenlet dependency for fidesctl[webserver]

[SteveDMurphy]

Bug Description

When starting up the webserver, an error is returned that greenlet is missing as a required dependency

Steps to Reproduce

1. pip install "fidesctl[webserver]"
2. fidesctl init
3. fidesctl webserver

Expected behavior

The webserver should successfully start, this can be resolved temporarily by:

1. pip install greenlet
2. fidesctl webserver

Screenshots

n/a

Environment

• Version: 1.7.0
• OS: Darwin
• Python Version: 3.9.12
• Docker Version: 20.10.14

Additional context

Discovered as part of working my way through fidesdemo, I'm not sure this is actually an issue with fidesctl but potentially a different dependency 🤷🏽

[sanders41]

My guess is this has to do with SqlAlchemy, but I am not sure why it wouldn't install on it's own

install_requires =
    importlib-metadata;python_version<"3.8"
    greenlet != 0.4.17;(platform_machine=='aarch64' or (platform_machine=='ppc64le' or (platform_machine=='x86_64' or (platform_machine=='amd64' or (platform_machine=='AMD64' or (platform_machine=='win32' or platform_machine=='WIN32'))))))
    typing-extensions >= 4.1.0

[SteveDMurphy]

Thanks for the lead @sanders41 ! Some secondary research made it seem like there may be something going on with one (or more) of our async calls that could be tripping this up as well...

[ThomasLaPiana]

note that #881 might affect this a bit, but I'll do some testing on this and see if I can't replicate it

[ThomasLaPiana]

pipx install fidesctl[webserver] -> fidesctl webserver worked fine for me on my local machine 🤔

given that I used pipx it should be totally isolated, so I'm not sure what might be causing this

[sanders41]

With pipx install fidesctl[webserver] -> fidesctl webserver I do get the error. @ThomasLaPiana what is your system Python version? Are you on 3.8.x? Mine is 3.10.5, and @SteveDMurphy has 3.9.12. If not that maybe it as to do with M1 Macs.

[sanders41]

Looks like it is an M1 issue. sqlalchemy/sqlalchemy#7714. The solution listed is to either add the asyncio extra to sqlalchemy, sqlchemy[asyncio] or to add greenlet to the requirements.

[ThomasLaPiana]

thanks for the heads up @sanders41 ! I'll go ahead and do the first option and document that its a fix