Skip to content

Commit

Permalink
Merge pull request #163 from sjenning/libvirt-vcpu-vars
Browse files Browse the repository at this point in the history
libvirt: hardcode memory and vcpus, boost vcpu default from 1 to 2
  • Loading branch information
openshift-merge-robot authored Aug 23, 2018
2 parents 27fb947 + beac99e commit 2f3cde6
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 21 deletions.
3 changes: 2 additions & 1 deletion steps/etcd/libvirt/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@ resource "libvirt_domain" "etcd" {
count = "${module.defaults.etcd_count}"

name = "etcd${count.index}"
memory = "${var.tectonic_libvirt_etcd_memory}"
memory = "1024"
vcpu = "2"
coreos_ignition = "${element(libvirt_ignition.etcd.*.id,count.index)}"

disk {
Expand Down
3 changes: 2 additions & 1 deletion steps/joining_workers/libvirt/workers.tf
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@ resource "libvirt_domain" "worker" {
count = "${var.tectonic_worker_count}"

name = "worker${count.index}"
memory = "${var.tectonic_libvirt_worker_memory}"
memory = "1024"
vcpu = "2"
coreos_ignition = "${libvirt_ignition.worker.id}"

disk {
Expand Down
3 changes: 2 additions & 1 deletion steps/masters/libvirt/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@ resource "libvirt_domain" "master" {

name = "master${count.index}"

memory = "${var.tectonic_libvirt_master_memory}"
memory = "2048"
vcpu = "2"

# Override ignition for the first (bootstrap) node. It can't be re-ignited,
# but that's okay for us
Expand Down
18 changes: 0 additions & 18 deletions steps/variables-libvirt.tf
Original file line number Diff line number Diff line change
Expand Up @@ -44,21 +44,3 @@ variable "tectonic_libvirt_first_ip_worker" {
description = "the offset in the ip range of the first worker"
default = "50"
}

variable "tectonic_libvirt_etcd_memory" {
type = "string"
description = "ram to allocate for each etcd node"
default = "1024"
}

variable "tectonic_libvirt_master_memory" {
type = "string"
description = "ram to allocate for each etcd node"
default = "2048"
}

variable "tectonic_libvirt_worker_memory" {
type = "string"
description = "ram to allocate for each etcd node"
default = "1024"
}

0 comments on commit 2f3cde6

Please sign in to comment.