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

Regression on version 1.5.x for ibm_cis_firewall #1472

Closed
pauljegouic opened this issue May 19, 2020 · 3 comments
Closed

Regression on version 1.5.x for ibm_cis_firewall #1472

pauljegouic opened this issue May 19, 2020 · 3 comments

Comments

@pauljegouic
Copy link

pauljegouic commented May 19, 2020

Hi there,

We are facing a regression with the resource ibm_cis_firewall that does not work with version 1.5.x

With 1.5.x :

Error:  cis_id or zone_id not passed

With 1.4.0:

ibm_cis_firewall.lockdown: Creating...
ibm_cis_firewall.lockdown: Creation complete after 2s [id=XXXXX:crn:v1:bluemix:public:internet-svcs:global:a/XXXX:XXXX-4dd5-4a55-bd53-XXXXX::]

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

Terraform template used for both test:

variable "region" {
  default = "eu-de"
}


provider ibm {
  region = var.region
  generation       = 1
  ibmcloud_timeout = 60
}

data "ibm_resource_group" "id" {
  name = "Default"
}

resource "ibm_cis_firewall" "lockdown" {
      cis_id        = "crn:v1:bluemix:public:internet-svcs:global:a/XXXXXXXXX"
      domain_id     = "XXXXXXXX"
      firewall_type = "lockdowns"

      lockdown {
          priority    = 1
          urls        = [
              "test.mydomain.com*"
          ]

          configurations {
              target = "ip"
              value  = "192.168.1.1"
          }
          configurations {
              target = "ip_range"
              value  = "168.78.82.5/32"
          }
      }
  }
@hkantare
Copy link
Collaborator

The cis firewall we changed to be inline with other CIS resources..where the domain_id should be combination of both cis_id and domain_id

@hkantare
Copy link
Collaborator

#1201

As part of this issue we changed the domain_id to be inline with all CIS resources...where its combination of cis_id/domain_id

@pauljegouic
Copy link
Author

Thanks ! I missed the changelog for 1.5.0 that mentionned it.

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

No branches or pull requests

2 participants