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

Add License Expiry Metric #15483

Open
wants to merge 3 commits into
base: devel
Choose a base branch
from
Open

Conversation

neevnuv
Copy link

@neevnuv neevnuv commented Aug 30, 2024

SUMMARY

Add a metric to show the time until the license for the awx/automation controller will expire.

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
  • Metrics
AWX VERSION
24.6.1
ADDITIONAL INFORMATION

This will add the expiry like describe in this issue, #3781


Copy link

Copy link

codecov bot commented Feb 4, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@neevnuv
Copy link
Author

neevnuv commented Feb 4, 2025

@djyasin can you rerun the pipeline for this PR? Plus I would love a review if possible

@djyasin
Copy link
Member

djyasin commented Feb 5, 2025

Hi @neevnuv! I have rerun the pipeline for your PR :)

@neevnuv
Copy link
Author

neevnuv commented Feb 7, 2025

Hi @neevnuv! I have rerun the pipeline for your PR :)

Thank you so much for running the pipeline. I have added the new metric to the test, so it won’t fail. Hopefully now they’ll be successfully

@neevnuv
Copy link
Author

neevnuv commented Feb 11, 2025

Hi @djyasin, I fixed the issue that caused the pipeline to fail. I would love if you could run it again

@neevnuv
Copy link
Author

neevnuv commented Feb 13, 2025

It seems all checks passed successfully :). This PR awaits review

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

Successfully merging this pull request may close these issues.

2 participants