Skip to content

Comments

resurrect r2#7124

Merged
boxp merged 2 commits intomainfrom
feature/resurrect-r2
Feb 21, 2026
Merged

resurrect r2#7124
boxp merged 2 commits intomainfrom
feature/resurrect-r2

Conversation

@boxp
Copy link
Owner

@boxp boxp commented Feb 21, 2026

No description provided.

@boxp-tfaction

This comment has been minimized.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: aef40226cd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

lifecycle {
destroy = false
}
resource "cloudflare_r2_bucket" "moltworker_data" {

Choose a reason for hiding this comment

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

P1 Badge R2復帰時にstate再関連付けを追加する

この cloudflare_r2_bucket.moltworker_data を戻した一方で、同ディレクトリには tfmigrate/20260220122258_rm_r2_bucket.hclrm cloudflare_r2_bucket.moltworker_data が残っているため、以前に state から外した環境で既存バケットがCloudflare側に残っている場合、importなしのまま apply すると同名バケット作成が衝突して失敗します。既存インフラを安全に再管理するには、この変更と同時に import か対応する state migration を追加する必要があります。

Useful? React with 👍 / 👎.

@boxp-tfaction
Copy link
Contributor

boxp-tfaction bot commented Feb 21, 2026

Plan Result (terraform/cloudflare/b0xp.io/moltworker)

CI link

Plan: 1 to add, 0 to change, 0 to destroy.
  • Create
    • cloudflare_r2_bucket.moltworker_data
Change Result (Click me)
  # cloudflare_r2_bucket.moltworker_data will be created
  + resource "cloudflare_r2_bucket" "moltworker_data" {
      + account_id = "1984a4314b3e75f3bedce97c7a8e0c81"
      + id         = (known after apply)
      + location   = "APAC"
      + name       = "moltbot-data"
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Warning: Deprecated Resource

  with cloudflare_access_application.moltworker,
  on access.tf line 3, in resource "cloudflare_access_application" "moltworker":
   3: resource "cloudflare_access_application" "moltworker" {

`cloudflare_access_application` is now deprecated and will be removed in the
next major version. Use `cloudflare_zero_trust_access_application` instead.

(and 2 more similar warnings elsewhere)

⚠️ Warnings

Warning: Deprecated Resource

  with cloudflare_access_application.moltworker,
  on access.tf line 3, in resource "cloudflare_access_application" "moltworker":
   3: resource "cloudflare_access_application" "moltworker" {

`cloudflare_access_application` is now deprecated and will be removed in the
next major version. Use `cloudflare_zero_trust_access_application` instead.

(and 2 more similar warnings elsewhere)

⚠️ Errors

  • failed to add a label terraform/cloudflare/b0xp.io/moltworker/add-or-update: label name is too long (max: 50)

@boxp boxp merged commit 69c4506 into main Feb 21, 2026
16 checks passed
@boxp boxp deleted the feature/resurrect-r2 branch February 21, 2026 01:48
@boxp-tfaction
Copy link
Contributor

boxp-tfaction bot commented Feb 21, 2026

❌ Apply Failed (terraform/cloudflare/b0xp.io/moltworker)

CI link

Error: Provider produced inconsistent result after apply

When applying changes to cloudflare_r2_bucket.moltworker_data, provider
"provider[\"registry.terraform.io/cloudflare/cloudflare\"]" produced an
unexpected new value: .location: was cty.StringVal("APAC"), but now
cty.StringVal("ENAM").

This is a bug in the provider, which should be reported in the provider's own
issue tracker.
Details (Click me)
Feb 21 01:49:20.969 WRN The policy file is ignored unless it is allowed by "aqua policy allow" command.

$ aqua policy allow "/home/runner/work/arch/arch/aqua/aqua-policy.yaml"

If you want to keep ignoring the policy file without the warning, please run "aqua policy deny" command.

$ aqua policy deny "/home/runner/work/arch/arch/aqua/aqua-policy.yaml"

  program=aqua version=2.56.6 exe_name=terraform package_name=hashicorp/terraform package_version=v1.14.5 registry=standard policy_file=/home/runner/work/arch/arch/aqua/aqua-policy.yaml doc=https://aquaproj.github.io/docs/reference/codes/003
cloudflare_r2_bucket.moltworker_data: Creating...

Error: Provider produced inconsistent result after apply

When applying changes to cloudflare_r2_bucket.moltworker_data, provider
"provider[\"registry.terraform.io/cloudflare/cloudflare\"]" produced an
unexpected new value: .location: was cty.StringVal("APAC"), but now
cty.StringVal("ENAM").

This is a bug in the provider, which should be reported in the provider's own
issue tracker.

@boxp-tfaction
Copy link
Contributor

boxp-tfaction bot commented Feb 21, 2026

terraform/cloudflare/b0xp.io/moltworker: Follow up PR was created

Build link

@boxp

Apply failed. Please handle the problem. ⚠️ Don't rerun GitHub Actions Workflow

  1. Check the error message
  2. Check chore(terraform/cloudflare/b0xp.io/moltworker): follow up #7124 #7125
  3. Add commits to chore(terraform/cloudflare/b0xp.io/moltworker): follow up #7124 #7125 to fix the problem if needed
  4. Review and merge chore(terraform/cloudflare/b0xp.io/moltworker): follow up #7124 #7125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant