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

This is a lot of things #6

Open
wants to merge 32 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
4999732
update resource group to variable so you can use a previously created…
Apr 7, 2021
fcf0f73
this is a wip
Apr 7, 2021
59e5122
update with changelog
Apr 8, 2021
2027600
servers working on apply
Apr 9, 2021
9c0597d
working with agents - location missing
Apr 9, 2021
6911144
working with agents - location variable
Apr 9, 2021
17cd200
change rg to variable in agents main
Apr 9, 2021
bc4c376
i was being stupid - subnet variable revert
Apr 9, 2021
557d955
add default values
Apr 9, 2021
92c0a4a
add default values
Apr 9, 2021
1d4301e
nodepool index on 0
Apr 9, 2021
29c85a8
adding types to all the agent vars
Apr 9, 2021
d42e342
more azure networking things
Apr 12, 2021
6617e53
sg names
Apr 12, 2021
6b6dee4
adding verbosity for testing
Apr 13, 2021
985c229
duh
Apr 13, 2021
d9ab348
add more output
Apr 13, 2021
5a0bae1
add more output init module
Apr 13, 2021
5392f62
add nattz
Apr 20, 2021
5aab407
remove tags
Apr 20, 2021
ea685d1
var name
Apr 20, 2021
fd046a8
subnet_id
Apr 20, 2021
1a995d0
begin making the statestore a real module
Apr 22, 2021
f3935ae
update todos
Apr 22, 2021
5293eb7
@joshwolf this is not aws
Apr 23, 2021
672932a
update init script for govcloud not commercial azure. This is the go…
Apr 29, 2021
19d292f
removing ridiculous outputs
Apr 29, 2021
1b9d951
all the things
Apr 29, 2021
2d4e3c6
finishing up
Apr 29, 2021
fba24f8
cloud provider @joshrwolf this one is azure
May 10, 2021
9fb197a
i hate azure
May 10, 2021
42dc067
Fixed a reference to an undeclared resource in the ssh resourcee
May 20, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
# Changelog

## 04/08/2021

### Branch: `patch-rgs`

* Remove the resource group definitions throughout modules and submodules and use separate resource group module to pass into new variables.
* Change `subnet_id` from `string` to `list(string)` to support `vnet` with multiple subnets.
* Add variable `location` to use location of the resource_group dependency
* Add variable `resource_group_id`

```hcl
resource "azurerm_lb_backend_address_pool" "bepool" {
name = "${var.name}-lbe-be-pool"
loadbalancer_id = azurerm_lb.this.id
resource_group_name = var.resource_group_name
# Deprecated in future azurerm releases, ignore linting
// resource_group_name = ""
}
```
In the `azurem_lb_backend_address_pool` resource, resource_group_name is required, so it was added back in.

* `upgrade_mode = "Manual"` (line 138 in `main.tf`) since `Automatic` was throwing errors about missing parameters not set.

8 changes: 8 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,11 @@ This repository is inteded to clearly demonstrate one method of deploying `rke2`

We highly recommend you use the modules in this repository as stepping stones in solutions that meet the needs of your workflow and organization. If you have suggestions or areas of improvements, we would [love to hear them](https://slack.rancher.io/)!


__WARNING:__ The leader election process is busted. To get this module to work you must select 1 server and rerun `01_rke.sh` in `/var/lib/cloud/instances/$INSTANCE/scripts/01_rke2.sh` on subsequent server nodes to get them to join the cluster.

The `agents` module, however works just fine in joining the cluster once a master is present.

## TODO:

* Figure out missing inputs to get `upgrade_mode = "Automatic"` working.
Loading