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

Doc updates #380

Closed
wants to merge 70 commits into from
Closed

Doc updates #380

wants to merge 70 commits into from

Conversation

james-otten
Copy link
Collaborator

@james-otten james-otten commented Jun 2, 2024

  • Figure out the meshdb-cluster stuff (first time deploy)
  • Finish docs
  • frr + haproxy

state: restarted
enabled: true
become: true

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add comment explaining the reasoning behind the steps?

Comment on lines 37 to 38
bind {{ EXTERNAL_LISTEN_IP }}:80
bind {{ EXTERNAL_LISTEN_IP }}:443
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be able to delete this, especially if we use traefik. Need to test.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

certbot used for let's encrypt instead of traefik

Comment on lines 13 to 14
nameservers:
addresses: [ "10.10.10.10", "10.10.10.11", "8.8.8.8" ]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might cause trouble; We'll see during the test.

infra/tf/mgr.tf Outdated Show resolved Hide resolved
@@ -66,7 +70,17 @@ variable "meshdb_networkrange" {
default = "24"
}

variable "meshdb_net_block" {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: Investigate this further

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a variable we use for FRR config.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cleaned up var names

@james-otten james-otten changed the base branch from wdn/infra to main July 16, 2024 02:54
Comment on lines +1 to +51
module "k3s" {
source = "xunleii/k3s/module"

depends_on_ = [
proxmox_vm_qemu.meshdbmgr,
proxmox_vm_qemu.meshdbnode,
]
k3s_version = "latest"
cluster_domain = "cluster.local"
cidr = {
pods = "10.42.0.0/16"
services = "10.43.0.0/16"
}
drain_timeout = "30s"
managed_fields = ["label", "taint"] // ignore annotations

global_flags = [
]

servers = {
for instance in proxmox_vm_qemu.meshdbmgr :
instance.name => {
ip = instance.default_ipv4_address
connection = {
host = instance.default_ipv4_address
# TODO: Try to use tls_private_key?
#private_key = trimspace(tls_private_key.ed25519_provisioning.private_key_pem)
private_key = file("${path.module}/meshdb${var.meshdb_env_name}")
user = "debian"
}
flags = [
"--write-kubeconfig-mode 644",
]
}
}

agents = {
for instance in proxmox_vm_qemu.meshdbnode :
instance.name => {
name = instance.name
ip = instance.default_ipv4_address
connection = {
host = instance.default_ipv4_address
# TODO: Try to use tls_private_key?
#private_key = trimspace(tls_private_key.ed25519_provisioning.private_key_pem)
private_key = file("${path.module}/meshdb${var.meshdb_env_name}")
user = "debian"
}
}
}
}

Check failure

Code scanning / checkov

Ensure Terraform module sources use a commit hash Error

Ensure Terraform module sources use a commit hash
@WillNilges
Copy link
Collaborator

Closing the following PRs in favor of this one:

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.

2 participants