Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Ephemeral parameter value not passed to provider when selected in "Build options" #8654

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

Closed
aaronlehmann opened this issue Jul 21, 2023 · 2 comments · Fixed by #8660
Closed
Assignees
Labels
site Area: frontend dashboard

Comments

@aaronlehmann
Copy link
Contributor

Trying out the new "ephemeral parameters" feature, I made this parameter in my template ephemeral:

data "coder_parameter" "rebuild_devcontainer" {
  name         = "rebuild_devcontainer"
  display_name = "Force devcontainer rebuild"
  type         = "bool"
  ephemeral    = true
  default      = "false"
  mutable      = true
  order        = 7
}

When I select the non-default value "true" for this parameter in the "Build Options" popup under "Restart", my provider's Update method still sees a false value in the plan.

image

However, this works as expected if I instead go to Settings->Parameters and change the value there.

cc @mtojek

@cdr-bot cdr-bot bot added the bug label Jul 21, 2023
@BrunoQuaresma BrunoQuaresma self-assigned this Jul 21, 2023
@mtojek
Copy link
Member

mtojek commented Jul 21, 2023

Smells like regression, Bruno.

Screenshot 2023-07-21 at 19 05 49

@BrunoQuaresma
Copy link
Collaborator

@mtojek Yes, but maybe it is not a regression because this happens when the user restarts the workspace and has to confirm the restart on the dialog. Since we usually have the "Don't ask me again" option enabled we probably missed this path during our QAs.

@BrunoQuaresma BrunoQuaresma added the site Area: frontend dashboard label Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site Area: frontend dashboard
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants