Skip to content

Commit

Permalink
Merge pull request #224 from pravin-dsilva/release-4.7-cni
Browse files Browse the repository at this point in the history
Add variable to set default CNI network provider
  • Loading branch information
ltccci authored Feb 26, 2021
2 parents c98d7d6 + 628b910 commit 5c23713
Show file tree
Hide file tree
Showing 7 changed files with 20 additions and 1 deletion.
6 changes: 6 additions & 0 deletions docs/var.tfvars-doc.md
Original file line number Diff line number Diff line change
Expand Up @@ -291,3 +291,9 @@ The following variables are specific to enable the connectivity between OCP node
ibm_cloud_dl_endpoint_net_cidr = ""
ibm_cloud_http_proxy = ""
```

This variable is used to set the default Container Network Interface (CNI) network provider such as OpenShiftSDN or OVNKubernetes

```
cni_network_provider = "OpenshiftSDN"
```
1 change: 1 addition & 0 deletions modules/5_install/install.tf
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,7 @@ locals {
squid_source_range = var.cidr
proxy_url = local.proxy.server == "" ? "" : "http://${local.proxy.user_pass}${local.proxy.server}:${local.proxy.port}"
no_proxy = var.cidr
cni_network_provider = var.cni_network_provider
}

powervs_config_vars = {
Expand Down
2 changes: 2 additions & 0 deletions modules/5_install/templates/install_vars.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,3 +41,5 @@ dhcp_shared_network: true
%{ if bastion_vip != "" }
bastion_vip: "${bastion_vip}"
%{ endif ~}

cni_network_provider: ${cni_network_provider}
2 changes: 2 additions & 0 deletions modules/5_install/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -87,3 +87,5 @@ variable "proxy" {}

variable "ibm_cloud_dl_endpoint_net_cidr" {}
variable "ibm_cloud_http_proxy" {}

variable "cni_network_provider" {}
1 change: 1 addition & 0 deletions ocp.tf
Original file line number Diff line number Diff line change
Expand Up @@ -122,4 +122,5 @@ module "install" {
upgrade_delay_time = var.upgrade_delay_time
ibm_cloud_dl_endpoint_net_cidr = var.ibm_cloud_dl_endpoint_net_cidr
ibm_cloud_http_proxy = var.ibm_cloud_http_proxy
cni_network_provider = var.cni_network_provider
}
2 changes: 2 additions & 0 deletions var.tfvars
Original file line number Diff line number Diff line change
Expand Up @@ -85,3 +85,5 @@ cluster_id = ""

#ibm_cloud_dl_endpoint_net_cidr = "" #Set this to IBM Cloud DirectLink endpoint network cidr eg. 10.0.0.0/8
#ibm_cloud_http_proxy = "" #Set this to IBM Cloud http/squid proxy eg. http://10.166.13.64:3128

#cni_network_provider = "OpenshiftSDN"
7 changes: 6 additions & 1 deletion variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ variable "install_playbook_tag" {
type = string
description = "Set the branch/tag name or commit# for using ocp4-playbooks repo"
# Checkout level for https://github.com/ocp-power-automation/ocp4-playbooks which is used for running ocp4 installations steps
default = "fc74d7ec06b2dd47c134c50b66b478abde32e295"
default = "592e51671ff2762718955fb2a0541a5b19c862e9"
}

variable "ansible_extra_options" {
Expand Down Expand Up @@ -455,6 +455,11 @@ variable "upgrade_delay_time" {
default = "600"
}

variable "cni_network_provider" {
description = "Set the default Container Network Interface (CNI) network provider"
default = "OpenshiftSDN"
}

################################################################
# Local registry variables ( used only for restricted network install )
################################################################
Expand Down

0 comments on commit 5c23713

Please sign in to comment.