Feat/dohrmy 139 dom

[pareddyakamai]

No description provided.

[jaescalo]

We should have a condition so that late validation is an option and not mandatory. A user can perform a pre-validation which wouldn't require this resource. Check out this example https://github.com/akamai/terraform-examples/blob/main/products/domain-ownership/late-validation/main.tf

[jaescalo]

@pareddyakamai I was referring more like a condition that would optionally enable the DOM post-validation flow. The reason is post-validation should also take into account the DNS validations, otherwise the activations will fail. Plus @rlopezso has a good point in which we could perform a check that the domains are validated prior to running the DOM module. What if we remove the post-validation flow for now and keep the pre-validation to avoid more delays.

[jaescalo]

@pareddyakamai we've decided to remove the late validation flow for now because of the DNS dependecies to validate the domains.

[rlopezso]

I agree with Jaime that this should be eligible and not mandated. But would it make more sense to add this to the DOM module? And through the deploy script, validate whether the domains are validated or not, and then call the dom module to perform the late validation?

[jaescalo]

@pareddyakamai this is related to removing the post validation flow for now.

[rlopezso]

Wonder why there are so many "dummy" changes to the Readme files?

[jaescalo]

@pareddyakamai I was referring more like a condition that would optionally enable the DOM post-validation flow. The reason is post-validation should also take into account the DNS validations, otherwise the activations will fail. Plus @rlopezso has a good point in which we could perform a check that the domains are validated prior to running the DOM module. What if we remove the post-validation flow for now and keep the pre-validation to avoid more delays.

[jaescalo]

Remove this empty line for the terraform-docs to add the documentation in this main.tf to the README.