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

c7n-salactus gives syntax error #7829

Open
haiauth opened this issue Oct 4, 2022 · 2 comments
Open

c7n-salactus gives syntax error #7829

haiauth opened this issue Oct 4, 2022 · 2 comments
Labels

Comments

@haiauth
Copy link

haiauth commented Oct 4, 2022

Describe the bug

I have installed the redis-server, get it running, installed supervisor and run the supervisord.conf file as well. But when I run c7n-salactus --help, I got a syntax error.

What did you expect to happen?

Well, I expect it to give me a help menu.

Cloud Provider

Amazon Web Services (AWS)

Cloud Custodian version and dependency information

0.9.18

Policy

No response

Relevant log/traceback output

Traceback (most recent call last):
  File "/home/ubuntu/custodian/bin/c7n-salactus", line 11, in <module>
    load_entry_point('c7n-salactus==0.3.0', 'console_scripts', 'c7n-salactus')()
  File "/home/ubuntu/custodian/lib/python3.8/site-packages/pkg_resources/__init__.py", line 489, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/home/ubuntu/custodian/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2852, in load_entry_point
    return ep.load()
  File "/home/ubuntu/custodian/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2443, in load
    return self.resolve()
  File "/home/ubuntu/custodian/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2449, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/home/ubuntu/custodian/lib/python3.8/site-packages/c7n_salactus/cli.py", line 251
    def reset(async=None):
              ^
SyntaxError: invalid syntax

Extra information or context

No response

@ajkerrigan
Copy link
Member

ajkerrigan commented Oct 4, 2022

Thanks for the report @haiauth 👍 . I'm seeing the same issue installing 0.3.0 from PyPI. Looks like that's dying because in current versions of Python, async is a keyword. However, it looks like #2608 fixed this issue in salactus back in 2018. Salactus isn't part of the set of typical c7n release packages, but this suggests we need to get a new release published. Given the time since last release, I wouldn't be surprised if we find other issues in the process.

@haiauth
Copy link
Author

haiauth commented Oct 4, 2022

@ajkerrigan Thank you for your response. Do you have a timeline when the new release/fix will be released?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants