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

feat: create queues and topics #61

Merged
merged 29 commits into from
Nov 26, 2024
Merged

feat: create queues and topics #61

merged 29 commits into from
Nov 26, 2024

Conversation

musifalsk
Copy link
Contributor

@musifalsk musifalsk commented Sep 26, 2024

No description provided.

musifalsk and others added 22 commits September 5, 2024 16:34
* chore: update required Azure provider version

Support for the `network_rule_set` block for resource `azurerm_servicebus_namespace` was added in Azure provider v3.71.

* refactor: rename variables

* refactor: remove variable for enabling network rules

* refactor: remove duplicate variable

* Fix configuration of Premium only features

* Deny public network access by default

* Dynamically configure network rules default action

* Rename variable

* Set optional object property

* Reorder blocks
@musifalsk musifalsk requested a review from a team as a code owner September 26, 2024 13:59
@hknutsen
Copy link
Member

Are these resources actually essential (as they are described in equinor/terraform-baseline#177), or does it make more sense for these resources to be created outside of this module? I think a good comparison would be our Azure Storage module which creates only the Storage account; containers, file shares, tables and queues must be created outside of the module.

@musifalsk
Copy link
Contributor Author

musifalsk commented Sep 30, 2024

without a queue or a Topic resource you only have a halfway completed resource that as far as i know can not be used for anything. You need at least 1 queue or 1 topic to forward messages to. But it might still be preferable to keep it outside?!

@hknutsen
Copy link
Member

hknutsen commented Oct 2, 2024

Suggestion: create queues, but not authorization rules, since authorization rules promote the use of access keys instead of RBAC.

If authorization rules are needed, create outside of module.

@musifalsk musifalsk changed the title feat: New resources: Queue and Authorization rules feat: New resources: Queue and Topic Nov 22, 2024
@hknutsen hknutsen changed the title feat: New resources: Queue and Topic feat: create queues and topics Nov 22, 2024
@hknutsen
Copy link
Member

Updated PR title.

variables.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
musifalsk and others added 3 commits November 25, 2024 12:50
Co-authored-by: Henrik Simonsen Knutsen <[email protected]>
Co-authored-by: Henrik Simonsen Knutsen <[email protected]>
@hknutsen hknutsen dismissed their stale review November 25, 2024 11:58

Changes implemented in separate commit.

Copy link
Member

@hknutsen hknutsen left a comment

Choose a reason for hiding this comment

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

Looks good 🎉

@musifalsk musifalsk merged commit 39aa8f5 into main Nov 26, 2024
2 checks passed
@musifalsk musifalsk deleted the service-bus-queue branch November 26, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants