Commit 18544db
release: 5.10.1 (#6164)
* fix: ruleset migration issues (#6163)
* Revert "fix: ruleset migration issues (#6163)" (#6165)
This reverts commit 44b653c.
* chore: enable mconn tests (#6166)
* fix: magic connector tests
* chore: remove skip
* fix: ruleset migration issues (#6168)
* feat: grit to go (#6162)
* feat: grit to go
* fix: unused import in load_balancer_monitor
* chore: remove debugging printf statement
---------
Co-authored-by: Musa Jundi <musa@cloudflare.com>
* fix: lb and lb pool config migration (#6170)
* fix: cloudflare_load_balancer transformation issues (#6171)
* Test improvements (#6172)
* chore: add sweeper for logpush job
* chore: define ci test product groups
* chore: limit max retries (#6173)
* fix: fix grit in migration tests (#6175)
* fix: ruleset migration in nogrit (#6174)
* chore: point transformations to gh/next (#6177)
* chore: zero trust config issues (#6179)
fix: fix zero trust access application state migration
* fix: lb monitor state migration (#6180)
* fix: lb monitor state
* fix: build
* fix(migrate): improve `zone_setting` migrations (#6169)
Improves how zone settings are migrated from v4 -> v5 of the provider by adding
two new flags:
- `--zone-settings-module` for handling a common pattern in the v4 provider
where `zone_settings_override` is wrapped in a module and settings are passed
in as module input variables. When provided, we will expand the vars into
`zone_setting` resources and imports at the call site.
- `--skip-imports` for skipping import generation, which is useful in cases
where imports could be generated in invalid locations (read: outside the root
module).
Module definition: `modules/zone_settings/main.tf`
```hcl
resource "cloudflare_zone_settings_override" "zone_settings" {
zone_id = var.zone_id
settings {
security_level = var.security_level
ssl = var.ssl
}
}
```
Module call: `sites/example_com/main.tf`
```hcl
module "zone_settings" {
source = "../modules/zone_settings"
zone_id = cloudflare_zone.example_com.id
security_level = "high"
ssl = "origin_pull"
}
```
When the `--zone-settings-module` flag is set, the migrator tool will replace
module calls with inline zone setting definitions and their imports.
```hcl
resource "cloudflare_zone_setting" "zone_settings_zone_settings_security_level" {
zone_id = cloudflare_zone.example_com.id
setting_id = "security_level"
value = "high"
}
resource "cloudflare_zone_setting" "zone_settings_zone_settings_ssl" {
zone_id = cloudflare_zone.example_com.id
setting_id = "ssl"
value = "origin_pull"
}
import {
to = cloudflare_zone_setting.zone_settings_zone_settings_security_level
id = "${cloudflare_zone.example_com.id}/security_level"
}
import {
to = cloudflare_zone_setting.zone_settings_zone_settings_ssl
id = "${cloudflare_zone.example_com.id}/ssl"
}
```
Import generation can be skipped entirely with the `--skip-imports` flag.
* fix: remove 'disable_railgun' from state after v4 migration (#6186)
* fix: handling of nested arrays in ruleset migration (#6187)
* fix: handling of nested arrays in ruleset migration
* chore: fix test data for transformation
* chore: compare better (#6192)
* fix: ruleset state (#6191)
* fix(zone_setting): ensure clean state after migrate (#6190)
* fix(zone_setting): ensure clean state after migrate
Fixes an issue with the migrate flow for `cloudflare_zone_setting` where we
weren't removing all settings from state after migrating from v4 -> v5. The
root cause was indexes shifting in slices while deleting items (forward deletes
vs backward deletes).
- updates state removal method for cleaning up `cloudflare_zone_setting`
resources to delete items in reverse
* test(zone_setting): ensure attribute order
Fixes an issue with migrated attribute order revealed in flaky tests.
---------
Co-authored-by: Vaishak Dinesh <vaishakpdinesh@gmail.com>
* fix: variable interpolation (#6193)
* fix: variable interpolation
* chore: missed page rule migrations test
* chore: restore deleted test cases
* fix(workers_script): fix incorect model type of `run_worker_first` attribute (#6199)
* fix: remove zone settings with null values (#6201)
* docs(list_item): add import documentation (#6202)
List item uses custom code for imports so its invisible to codegen. Since docs
are generated automatically, any custom docs changes will be ovewritten with
every release.
This patch updates the generator script to first create any missing examples
that couldn't be codegen'd, then it runs `tfplugindocs` which ensures that
import examples are appended to the generated markdown docs.
Changes:
- adds manual `import.sh` for `cloudflare_list_item`
* fix(migrate): block transformations (#6203)
* Revert "fix: variable interpolation (#6193)"
This reverts commit 332de8d.
* fix(migrate): block transformation
* fix(migrate): lb and lb pools
* fix(migrate): fix main_module value migration (#6204)
* ci(page_rule): disable parallel tests (#6205)
Fixes flaky `page_rule` tests in CI. Parallel tests causes issues with it's
strange `priority` behavior.
Changes:
- set `parallel=1` for `page_rule` tests
* ci: skip flaky tests (#6206)
Skipping consistently failing tests due to cert conflicts.
- `TestMigrateZeroTrustAccessMTLSCertificate_Basic`
- `TestAccCloudflareAccessMutualTLSHostnameSettings_Account`
- `TestAccCloudflareAccessMutualTLSHostnameSettings_Update`
- `TestAccCloudflareAccessMutualTLSHostnameSettings_BooleanCombinations`
- `TestAccCloudflareAccessMutualTLSHostnameSettings_Import`
* release: 5.10.1
---------
Co-authored-by: Musa Jundi <musa@cloudflare.com>
Co-authored-by: Vaishak Dinesh <vaishakpdinesh@gmail.com>
Co-authored-by: Tamás Józsa <jtomi92@gmail.com>
Co-authored-by: Michael Girouard <206137+mgirouard@users.noreply.github.com>
Co-authored-by: Cina Saffary <cina@cloudflare.com>
Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>1 parent 7877ed3 commit 18544db
File tree
92 files changed
+13247
-694
lines changed- .github/workflows
- cmd/migrate
- transformations
- config
- docs/resources
- examples/resources/cloudflare_list_item
- internal
- acctest
- services
- load_balancer_monitor
- load_balancer_pool
- load_balancer
- logpush_job
- magic_transit_connector
- ruleset
- testdata/migrations
- workers_script
- zero_trust_access_application
- zero_trust_access_mtls_certificate
- zero_trust_access_mtls_hostname_settings
- scripts
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
92 files changed
+13247
-694
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
| 41 | + | |
41 | 42 | | |
42 | 43 | | |
43 | 44 | | |
44 | 45 | | |
| 46 | + | |
45 | 47 | | |
46 | 48 | | |
47 | 49 | | |
48 | 50 | | |
| 51 | + | |
49 | 52 | | |
50 | 53 | | |
| 54 | + | |
51 | 55 | | |
52 | 56 | | |
53 | | - | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
54 | 60 | | |
55 | | - | |
56 | | - | |
57 | | - | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
58 | 70 | | |
59 | 71 | | |
| 72 | + | |
60 | 73 | | |
61 | | - | |
62 | | - | |
63 | | - | |
64 | | - | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
65 | 79 | | |
66 | 80 | | |
| 81 | + | |
67 | 82 | | |
68 | | - | |
69 | | - | |
70 | | - | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
71 | 87 | | |
72 | 88 | | |
| 89 | + | |
73 | 90 | | |
| 91 | + | |
74 | 92 | | |
75 | | - | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
76 | 98 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
| 2 | + | |
3 | 3 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
3 | 47 | | |
4 | 48 | | |
5 | 49 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
105 | 105 | | |
106 | 106 | | |
107 | 107 | | |
108 | | - | |
| 108 | + | |
109 | 109 | | |
110 | 110 | | |
111 | 111 | | |
| |||
169 | 169 | | |
170 | 170 | | |
171 | 171 | | |
172 | | - | |
| 172 | + | |
173 | 173 | | |
174 | 174 | | |
175 | 175 | | |
| |||
299 | 299 | | |
300 | 300 | | |
301 | 301 | | |
302 | | - | |
| 302 | + | |
303 | 303 | | |
304 | 304 | | |
305 | 305 | | |
| |||
372 | 372 | | |
373 | 373 | | |
374 | 374 | | |
375 | | - | |
| 375 | + | |
376 | 376 | | |
377 | 377 | | |
378 | 378 | | |
| |||
454 | 454 | | |
455 | 455 | | |
456 | 456 | | |
457 | | - | |
| 457 | + | |
458 | 458 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
143 | 143 | | |
144 | 144 | | |
145 | 145 | | |
146 | | - | |
| 146 | + | |
147 | 147 | | |
0 commit comments