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

Feature/initial ipam #19

Open
wants to merge 137 commits into
base: main
Choose a base branch
from
Open

Conversation

ChrisChapman-gh
Copy link

Description

Initial commit of azure ipam pattern

Type of Change

  • Non-module change (e.g. CI/CD, documentation, etc.)
  • Azure Verified Module updates:
    • Bugfix containing backwards compatible bug fixes
      • Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
      • The bug was found by the module author, and no one has opened an issue to report it yet.
    • Feature update backwards compatible feature updates, and I have bumped the MINOR version in locals.version.tf.json.
    • Breaking changes and I have bumped the MAJOR version in locals.version.tf.json.
    • Update to documentation

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
  • I did run all pre-commit checks

microsoft-github-operations bot and others added 30 commits August 29, 2023 10:34
* Create CODEOWNERS

* Update CODEOWNERS

* Update CODEOWNERS

* Update CODEOWNERS
* Update avmrequiredfiles.yml

* Update avmrequiredfiles.yml

* Update CODEOWNERS
* feat: structure

* add tests dir

* test add .go file
* Update CODEOWNERS

* Update CODEOWNERS
…zure#17)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update main.telemetry.tf

renamed telemetry resource name to be consistent with local variables and removed location argument as it is not needed

* updated readme
matt-FFFFFF and others added 25 commits July 22, 2024 08:40
* Update eventResponder.yml

* Update scheduledSearches.yml
* Add new e2e test templates

* Check for fork

* Debugging

* Bug fixing

* Fixes

* Rename stuff

* Linting

* Overwrite the e2e file

* Add fork warning message

* Add warning

* Remove warning and add check summary

* Add message to summary

* Remove condition

* debug

* Inherit secrets
The goal is of this volume mount was to translate usernames from ids. However this is not really necessary, and exposes the full content of the passwd and group file from the host to the container.
Moreover, on systems like MacOS where docker runs into an internal Linux VM, the content of the file /etc/passwd is completely irrelevant.
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.7 to 4.2.2.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@692973e...11bd719)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
add locals.version
remove leftover template file
@ChrisChapman-gh ChrisChapman-gh requested a review from a team as a code owner November 10, 2024 20:01
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 Maintainers need to triage still label Nov 10, 2024
@ChrisChapman-gh
Copy link
Author

@microsoft-github-policy-service agree [company="bjss"]

@ChrisChapman-gh
Copy link
Author

@microsoft-github-policy-service agree company="bjss"

@mbilalamjad mbilalamjad linked an issue Dec 11, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Triage 🔍 Maintainers need to triage still
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AVM-Review