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

Bicep updates for validation and clean up #161

Closed
wants to merge 4 commits into from
Closed

Bicep updates for validation and clean up #161

wants to merge 4 commits into from

Conversation

BernieWhite
Copy link

@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
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
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
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
Contributor

Will be introduced once the work surrounding #1624 is completed

@BernieWhite
Copy link
Author

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

@AlexanderSehr
Copy link
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
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
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