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 missing quota tags #18656

Merged
merged 2 commits into from
Jul 17, 2019
Merged

Conversation

jeffmcutter
Copy link
Contributor

Automate quota validation is designed to be able to make use of the following tags / tag categories, but they do not exist by default:

Quota - Max VMs

Quota - Warn CPUs
Quota - Warn Memory
Quota - Warn Storage
Quota - Warn VMs

@miq-bot
Copy link
Member

miq-bot commented Apr 14, 2019

Checked commits jeffmcutter/manageiq@dea4a4a~...c91a13d with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@jeffmcutter
Copy link
Contributor Author

@tinaafitz

Can you comment on this? Does it make sense?

Thanks,
-Jeff

Copy link
Member

@tinaafitz tinaafitz left a comment

Choose a reason for hiding this comment

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

Thanks @jeffmcutter for adding the missing categories and tags.

@tinaafitz
Copy link
Member

@lfu Please review.

- :description: 1GB
:read_only: "0"
:syntax: string
:name: "1024"
Copy link
Member

Choose a reason for hiding this comment

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

Why is the name not set the same way as for Quota - Warn Storage?

Copy link
Member

Choose a reason for hiding this comment

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

@lfu Unfortunately, tags values differ for storage and memory. The tag values from max_storage/warn_storage should match, as well as values for max_memory/warn_memory.

Copy link
Member

Choose a reason for hiding this comment

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

We can't change the values because it would break existing customers environments.

@tinaafitz
Copy link
Member

@bdunne Please review.

@jeffmcutter
Copy link
Contributor Author

Hi @tinaafitz @bdunne,

Did this get lost or is there an issue?

Thanks,
-Jeff

@bdunne bdunne merged commit ea1b832 into ManageIQ:master Jul 17, 2019
@bdunne bdunne added this to the Sprint 116 Ending Jul 22, 2019 milestone Jul 17, 2019
@bdunne bdunne self-assigned this Jul 17, 2019
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.

5 participants