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/Network Security Group #589

Open
wants to merge 66 commits into
base: master
Choose a base branch
from
Open

Feat/Network Security Group #589

wants to merge 66 commits into from

Conversation

cbeti-ionos
Copy link
Member

What does this fix or implement?

Implements Terraform resources for Network Security Groups and Network Security Group Rules

Checklist

  • PR name added as appropriate (e.g. feat:/fix:/doc:/test:/refactor:)
  • Tests added or updated
  • Documentation updated
  • Changelog updated and version incremented (label: upcoming release)
  • Github Issue linked if any
  • Jira task updated

cbeti-ionos and others added 30 commits June 4, 2024 09:17
* feat: add nsg rule resource, moved logic from nsg
# Conflicts:
#	go.mod
#	ionoscloud/data_source_cube_server.go
#	ionoscloud/data_source_nic.go
#	ionoscloud/data_source_server.go
#	ionoscloud/provider.go
#	ionoscloud/resource_cube_server.go
#	ionoscloud/resource_nic.go
#	ionoscloud/resource_nic_test.go
#	ionoscloud/resource_server_test.go
#	services/cloudapi/cloudapinic/nic.go
#	utils/constant/constants.go
docs/data-sources/nsg.md Outdated Show resolved Hide resolved
docs/data-sources/nsg.md Outdated Show resolved Hide resolved
- [ionoscloud_nsg](docs/resources/nsg.md)
- [ionoscloud_nsg_firewallrule](docs/resources/nsg_firewallrule.md)
- `Data Sources`:
- [ionoscloud_nsg](docs/data-sources/nsg.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

@cristiGuranIonos shouldn't we also have a DS for NSG Firewall Rule?. I see that we can retrieve a NSG using the DS and look into the firewall rules for that NSG, but maybe we also want to retrieve a specific firewall rule using the DC ID and the NSG ID.

docs/data-sources/networkloadbalancer.md Show resolved Hide resolved
docs/data-sources/nsg.md Outdated Show resolved Hide resolved
docs/data-sources/nsg.md Outdated Show resolved Hide resolved
docs/data-sources/nsg.md Outdated Show resolved Hide resolved
ionoscloud/resource_nsg_firewallrule.go Outdated Show resolved Hide resolved
services/cloudapi/nsg/nsg.go Outdated Show resolved Hide resolved
services/cloudapi/nsg/nsg.go Outdated Show resolved Hide resolved
services/cloudapi/nsg/nsg.go Outdated Show resolved Hide resolved
services/cloudapi/nsg/nsg.go Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Nov 8, 2024

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.

4 participants