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

[Bug]: Can't configure a value for "flow_status" on AppFlow Flows #36441

Open
gbeaven-uturn opened this issue Mar 18, 2024 · 3 comments · May be fixed by #37501
Open

[Bug]: Can't configure a value for "flow_status" on AppFlow Flows #36441

gbeaven-uturn opened this issue Mar 18, 2024 · 3 comments · May be fixed by #37501
Labels
bug Addresses a defect in current functionality. service/appflow Issues and PRs that pertain to the appflow service.

Comments

@gbeaven-uturn
Copy link

Terraform Core Version

1.5.0

AWS Provider Version

5.41.0

Affected Resource(s)

aws_appflow_flow

Expected Behavior

AppFlow Flow status should be configurable in the resource block.

Relating to this issue #34574: when an AppFlow Flow is pushed with trigger_type="Scheduled" it was deploying in AWS with a status of "Draft", not "Active" - requiring an additional step either through the UI or API to update the flow to a status of "Active" before the flow could be used.

Actual Behavior

A fix was subsequently implemented on #34948 addressing the original issue.

However, the merged fix appears to have no affect on the behavior - Flows are still deployed in a "Draft" state requiring additional interaction in the UI or API before the flow is useable. The documentation states that a "flow_status" attribute reference is exported, but makes no reference to this field as an argument in the resource block.

Relevant Error/Panic Output Snippet

Error: Value for unconfigurable attribute
│ 
│   with module.appflow_flows.aws_appflow_flow.flow,
│   on main.tf line 31, in resource "aws_appflow_flow" "flow":
│   31:   flow_status = "Active"
│ 
│ Can't configure a value for "flow_status": its value will be decided automatically based on the result of applying this configuration.

Terraform Configuration Files

resource "aws_appflow_flow" "flow" {
  name       = "my-appflow-name"
  flow_status = "Active"
  ...
  trigger_config {
        trigger_type="Scheduled"
        trigger_properties{
            scheduled {
                schedule_expression  = "cron(0 1 * * ? *)"
            }
        }
    }
}

Steps to Reproduce

  1. Deploy resource without a status argument
  2. Observe "Draft" value on the "Flow status" field in UI:
    image

  1. Deploy resource with flow_status = "Active" status argument
  2. Observe error message:

Can't configure a value for "flow_status": its value will be decided automatically based on the result of applying this configuration.

Debug Output

No response

Panic Output

No response

Important Factoids

No response

References

No response

Would you like to implement a fix?

None

@gbeaven-uturn gbeaven-uturn added the bug Addresses a defect in current functionality. label Mar 18, 2024
Copy link

Community Note

Voting for Prioritization

  • Please vote on this issue by adding a 👍 reaction to the original post to help the community and maintainers prioritize this request.
  • Please see our prioritization guide for information on how we prioritize.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

Volunteering to Work on This Issue

  • If you are interested in working on this issue, please leave a comment.
  • If this would be your first contribution, please review the contribution guide.

@github-actions github-actions bot added the service/appflow Issues and PRs that pertain to the appflow service. label Mar 18, 2024
@terraform-aws-provider terraform-aws-provider bot added the needs-triage Waiting for first response or review from a maintainer. label Mar 18, 2024
@justinretzolk
Copy link
Member

Hey @gbeaven90 👋 Thank you for taking the time to raise this! With the way that flow_status is currently implemented, it is a read-only attribute, which is why you're receiving the error that you are. That said, it's unclear to me how the pull request that you mentioned was intended to fix the larger issue, so I'd like to leave this issue open for further review and prioritization.

@justinretzolk justinretzolk removed the needs-triage Waiting for first response or review from a maintainer. label Mar 20, 2024
@Viriathus1 Viriathus1 linked a pull request May 14, 2024 that will close this issue
@Viriathus1
Copy link
Contributor

Hey @geoff-beaven I had the same issue and it's a needed fix.
Can you 👍 my open pr #37501 which will resolve it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Addresses a defect in current functionality. service/appflow Issues and PRs that pertain to the appflow service.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants