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

Create html based report for dry runs for cloud providers. #138

Merged
merged 6 commits into from
Oct 3, 2024

Conversation

msajidmansoori12
Copy link
Contributor

This PR adds functionality to create html based reports for dry runs for AWS, AZURE, GCP.

Sample report attached.
Screenshot 2024-07-10 at 16-16-11 Cloud resources page (1)

@jyejare jyejare added Develop Branch RFE New Feature Request labels Aug 5, 2024
Copy link
Collaborator

@jyejare jyejare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would love to see Resource Types being on Rows and the data of them in the next column.

cloudwash/utils.py Outdated Show resolved Hide resolved
logger.info("Resources eligible for cleanup:")
for key, value in resource_data.items():
if value and key != "provider":
logger.info(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is repeating Deletable VMs with VMs (or any resource type), So its good to keep the old format of:

=========== DRY SUMMARY ============

VMs:
	Deletable: ['test-bvhoduliam']
	Stoppable: ['foremanqe-nightly2']

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, refactored the code to print the same way.

@msajidmansoori12
Copy link
Contributor Author

Screenshot 2024-10-03 at 12-52-58 Cloud resources page
After incorporating the comments , the output turns out to be like the table above.

@jyejare jyejare merged commit 8118a3e into RedHatQE:develop Oct 3, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Develop Branch RFE New Feature Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants