ansible-prometheus
ansible-prometheus copied to clipboard
Remove unnecessary dependency on jmespath
This is fairly straightforward manipulation to do with standard Jinja features, there's no need to use a non-default library.
upvote, let's wait and see, tbh, i gotta manually add ansible.builtin.pip for this jmespath in my playbook, what a pain. haha
This needs a rebase to get the tests to pass.
Or, maybe something else is broken. :frowning_face:
Ahh, I think I see what's going on. molecule is not creating the role symlink because it's a PR from a fork. Boo.
Or, maybe something else is broken.
This is a brief window of time during which the latest newest ansible-compat is broken.
Change test-requirements.txt:
# temporarily lock versions of molecule and ansible-compat to avoid a bug:
ansible-compat==0.5.0
molecule==3.5.2
Ok, fixed HEAD. Please rebase again.
@SuperQ Looking a little better, but it's still complaining about the Mergify configuration, and I'm not sure what's up with the failure for test-2.9-default.
- mergify strict mode is end-of-life
- circleci config file, should run on large instances. resource_class: large
@sdarwin Thanks, looks like we need to update the Mergify config in the auto-maintenance repo.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This role has been deprecated in favor of a the prometheus-community/ansible collection.