Skip to content

Conversation

@agustincelentano
Copy link
Collaborator

Summary

  • Remove configuration_aliases from OCI modules (dns, dynamic_groups, vcn) to simplify provider setup
  • Keep configuration_aliases in OKE module (requires oci.home for identity operations)
  • Add 5-minute timeouts to oci_dns_zone resource to prevent premature failure
  • Bump cert_manager_config_version default to 2.34.0
  • Bump nullplatform_base_helm_version default to 2.35.0 (includes OCI/OKE gateway support, cert-manager OCI support, and security fixes)

Test plan

  • tofu validate passes on all modules
  • tofu apply successful on infrastructure/, nullplatform/, and nullplatform-bindings/ in OCI implementation
  • DNS zones created successfully with new timeouts
  • Helm charts deployed with updated versions

Agustin Celentano added 4 commits February 3, 2026 12:01
Use single OCI provider instead of requiring explicit provider
mapping with configuration_aliases. The OKE module internally
maps oci.home = oci for the upstream oracle-terraform-modules/oke
dependency.
The upstream oracle-terraform-modules/oke requires oci.home alias,
so the wrapper must keep configuration_aliases for OKE. Other OCI
modules (vcn, dns, dynamic_groups) no longer require it.
@violenti violenti force-pushed the fix/remove-oci-configuration-aliases branch from 9b5a58d to 6e1ff98 Compare February 3, 2026 15:01
@violenti violenti merged commit 2e65a28 into main Feb 3, 2026
66 of 68 checks passed
@violenti violenti deleted the fix/remove-oci-configuration-aliases branch February 3, 2026 15:06
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.

3 participants