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

variable name missmatch #61

Open
chboudry opened this issue Feb 27, 2023 · 4 comments · May be fixed by #74
Open

variable name missmatch #61

chboudry opened this issue Feb 27, 2023 · 4 comments · May be fixed by #74

Comments

@chboudry
Copy link

Github action is expecting :

  • TEST_APP_ID, does not exist in the doc, and I think this is test_account_client_id in config.yaml ?
  • TEST_WORKSPACE_APP_ID, the doc says it is WORKSPACE_API_CLIENT_ID in the yaml, but that is not there, must be api_client_id ?
  • TEST_WORKSPACE_APP_SECRET, the docs says it is WORKSPACE_API_CLIENT_SECRET in the yaml, but it does not exist, must be api_client_secret ?
@LizaShak
Copy link
Contributor

LizaShak commented Mar 6, 2023

Hi @chboudry , thank you for raising the issue. We will look into that.

@SvenAelterman
Copy link
Contributor

Same experience here.

@chboudry Did you attempt a workaround?

@LizaShak
Copy link
Contributor

LizaShak commented Apr 3, 2023

@SvenAelterman If you are using an automation account, the nTEST_APP_ID is redundant. Same for TEST_WORKSPACE_APP_ID and TEST_WORKSPACE_APP_SECRET. We have an open issue to make it a non required params. For now you can put any dummy value inside (non empty value). Thank you and sorry for inconvenience.

@marrobi
Copy link
Member

marrobi commented Apr 21, 2023

@LizaShak was an issue created for this? It has come up a couple of times recently. Thanks.

@t-young31 t-young31 linked a pull request Jun 6, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants