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

Warning lines for instances not in inventory #29

Closed
rrossouw01 opened this issue May 20, 2022 · 2 comments
Closed

Warning lines for instances not in inventory #29

rrossouw01 opened this issue May 20, 2022 · 2 comments

Comments

@rrossouw01
Copy link

This warning can be quite annoying if you have many instances not in inventory. It shows not just on a list but also ssh-ssm utils etc

$ ssm-session --list --profile admin-guideitprod
[ssm-session] WARNING: SSM inventory entity not recognised: {'Id': 'i-02a#############c4b', 'Data': {}}
....

I changed the warning to debug temporarily maybe you want to make that permanent. Or if better option let me know.

$ diff /usr/local/lib/python3.8/dist-packages/ssm_tools/resolver.py /tmp/resolver.py
39a40
>
61,62c62
<                     #logger.warning("SSM inventory entity not recognised: %s", entity)
<                     logger.debug("SSM inventory entity not recognised: %s", entity)
---
>                     logger.warning("SSM inventory entity not recognised: %s", entity)
@mludvig
Copy link
Owner

mludvig commented Jun 25, 2022

Fixed in v1.4.1, thanks for the report.

I'm intrigued when does it happen though? I've got instances that are in SSM as well as many that are not and have never seen this error myself.

@mludvig mludvig closed this as completed Jun 25, 2022
@arthurio
Copy link
Contributor

@mludvig I have no idea what makes some instances show up in that list of "SSM inventory entity not recognised" either but I do see quite a few that are from a separate EKS cluster (no ssm installed on them), but I don't see "all" of them which doesn't really make sense ¯_(ツ)_/¯ And they are also terminated instances. Anyway, thanks for the log level change here 👍🏻

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

No branches or pull requests

3 participants