Skip to content

feat: support for Azure quota rule#224

Merged
mattwelke merged 5 commits intomainfrom
feat/azure-quota-rule
Sep 11, 2024
Merged

feat: support for Azure quota rule#224
mattwelke merged 5 commits intomainfrom
feat/azure-quota-rule

Conversation

@mattwelke
Copy link
Contributor

@mattwelke mattwelke commented Sep 11, 2024

Issue

Closes #225.

Description

Adds support for the rule. Like other rules with potentially repetitive input, users can provide input with a JSON file too.

Extras

  • Fixes a bug where if a user said they didn't want to enter Azure community image gallery rules, it would force them to anyways.
  • Fixes a bug where when users were prompted for a permission set for an Azure RBAC rule, the prompt incorrectly called it an IAM (for AWS) rule.

@mattwelke mattwelke requested a review from a team as a code owner September 11, 2024 18:30
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working new-feature Net-new feature labels Sep 11, 2024
Matt Welke added 2 commits September 11, 2024 14:34
Signed-off-by: Matt Welke <matt.welke@spectrocloud.com>
Signed-off-by: Matt Welke <matt.welke@spectrocloud.com>
@mattwelke mattwelke force-pushed the feat/azure-quota-rule branch from 54c4682 to 215f882 Compare September 11, 2024 18:35
Copy link
Member

@TylerGillson TylerGillson left a comment

Choose a reason for hiding this comment

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

Since you have fresh context, can you please take a look at #29 and either address it as part of this PR or as a follow up?

Signed-off-by: Matt Welke <matt.welke@spectrocloud.com>
@codecov
Copy link

codecov bot commented Sep 11, 2024

Codecov Report

Attention: Patch coverage is 42.22222% with 52 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/services/validator/azure.go 37.34% 36 Missing and 16 partials ⚠️
@@            Coverage Diff             @@
##             main     #224      +/-   ##
==========================================
- Coverage   53.30%   53.13%   -0.18%     
==========================================
  Files          46       46              
  Lines        6352     6433      +81     
==========================================
+ Hits         3386     3418      +32     
- Misses       2117     2151      +34     
- Partials      849      864      +15     
Files with missing lines Coverage Δ
pkg/config/constants.go 100.00% <ø> (ø)
...tegration/validatorctl/testcases/test_validator.go 93.41% <100.00%> (+0.06%) ⬆️
pkg/services/validator/azure.go 39.57% <37.34%> (-1.21%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 466413a...d4fc6dd. Read the comment docs.

…ure RBAC and quota rules.

Signed-off-by: Matt Welke <matt.welke@spectrocloud.com>
@mattwelke
Copy link
Contributor Author

mattwelke commented Sep 11, 2024

Since you have fresh context, can you please take a look at #29 and either address it as part of this PR or as a follow up?

Sure, I'll work on that next and add it to this PR as an extra bug fix do that in a follow up.

@TylerGillson TylerGillson self-requested a review September 11, 2024 19:04
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 11, 2024
@mattwelke mattwelke merged commit 8e61091 into main Sep 11, 2024
@mattwelke mattwelke deleted the feat/azure-quota-rule branch September 11, 2024 20:46
TylerGillson added a commit that referenced this pull request Sep 12, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.2.1](v0.2.0...v0.2.1)
(2024-09-11)


### Features

* support for Azure quota rule
([#224](#224))
([8e61091](8e61091))


### Bug Fixes

* ignore nil errors in validationResponseOk
([#227](#227))
([26544e2](26544e2))


### Refactoring

* split ConfigureOrCheckCommand into multiple functions
([#222](#222))
([466413a](466413a))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer new-feature Net-new feature size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ Add support for Azure quota rule

2 participants