Skip to content

Bicep updates for validation and clean up#161

Closed
BernieWhite wants to merge 4 commits intoAzure:mainfrom
BernieWhite:bewhite/add-ps-rule-validation
Closed

Bicep updates for validation and clean up#161
BernieWhite wants to merge 4 commits intoAzure:mainfrom
BernieWhite:bewhite/add-ps-rule-validation

Conversation

@BernieWhite
Copy link
Copy Markdown

@BernieWhite BernieWhite commented Oct 11, 2021

Change

  • Updates to bicep files and straggling parameter files.
  • Added tweaks to parameter validation and docs.
  • Added example for PSRule integration as WAF aligned tests.

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

@BernieWhite
Copy link
Copy Markdown
Author

@segraef Something to get the process started.

@BernieWhite BernieWhite marked this pull request as ready for review October 11, 2021 15:44
Copy link
Copy Markdown
Contributor

@AlexanderSehr AlexanderSehr left a comment

Choose a reason for hiding this comment

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

All changes for the ARM templates can be ignored/removed as those templates will be removed within the next week(s). The same is true for anything pointing at them (aka references).

All other parts (like the analysis) look good. In any case thanks for the updates, it just happens that the repo will only contain bicep, but not ARM templates.

@BernieWhite BernieWhite changed the title Add initial validation with PSRule for Azure Bicep updates for validation and clean up Oct 13, 2021
@rahalan rahalan added this to the MVP milestone Oct 21, 2021
@AlexanderSehr AlexanderSehr marked this pull request as draft October 25, 2021 13:14
@BernieWhite BernieWhite marked this pull request as ready for review October 27, 2021 15:35
@BernieWhite
Copy link
Copy Markdown
Author

@MrMCake Let me know if you want to chat on the updates.

@AlexanderSehr AlexanderSehr modified the milestones: MVP, Backlog Oct 29, 2021
@AlexanderSehr AlexanderSehr added enhancement New feature or request bug Something isn't working labels Oct 29, 2021
@BernieWhite BernieWhite marked this pull request as draft November 8, 2021 11:27
@BernieWhite BernieWhite marked this pull request as ready for review March 30, 2022 13:00
@rahalan rahalan removed this from the Backlog milestone Mar 30, 2022
@AlexanderSehr AlexanderSehr added the blocked if an issue is blocked label Aug 8, 2022
@AlexanderSehr
Copy link
Copy Markdown
Contributor

Will be introduced once the work surrounding #1624 is completed

@BernieWhite
Copy link
Copy Markdown
Author

@MrMCake Is this something that we can reactivate now #1624 is done, or are we waiting for #1787 as well?

@AlexanderSehr
Copy link
Copy Markdown
Contributor

AlexanderSehr commented Nov 7, 2022

Hey @BernieWhite please excuse the very late reply - have always had your PR in the back of my head as we get closer to enabling its implementation (finally). So we are we at now:

As far as I'm concerned, this will be our next big focus area once 1791 is merged - and any guidance you can provide is naturally more than welcome. If I understood @eriqua correctly, it will be some work to both finetune the rules for CARML (as some rules, for example, apply better to solutions rather than modules) and also address any findings. :)

@MariusStorhaug MariusStorhaug requested a review from a team as a code owner February 2, 2023 12:14
@AlexanderSehr AlexanderSehr marked this pull request as draft March 11, 2023 21:46
@eriqua
Copy link
Copy Markdown
Contributor

eriqua commented Apr 3, 2023

@BernieWhite PSRule has finally been integrated with GitHub workflows. Ref:

Currently this is implemented in audit mode. Meanwhile, we're working on module tests to adhere with the rules. Ref:

This was very much thanks to this draft PR, which helped us getting the base of the implementation.

@BernieWhite @AlexanderSehr if you agree I would then suggest to close this PR.

@BernieWhite
Copy link
Copy Markdown
Author

@BernieWhite PSRule has finally been integrated with GitHub workflows. Ref:

Currently this is implemented in audit mode. Meanwhile, we're working on module tests to adhere with the rules. Ref:

This was very much thanks to this draft PR, which helped us getting the base of the implementation.

@BernieWhite @AlexanderSehr if you agree I would then suggest to close this PR.

@eriqua Great work. Yep. Sounds good to me.

@BernieWhite BernieWhite closed this Apr 3, 2023
@BernieWhite BernieWhite deleted the bewhite/add-ps-rule-validation branch April 3, 2023 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked if an issue is blocked bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants