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

Bad error message for 'rad bicep publish' without tag #6296

Open
rynowak opened this issue May 24, 2023 · 3 comments
Open

Bad error message for 'rad bicep publish' without tag #6296

rynowak opened this issue May 24, 2023 · 3 comments
Labels
good first issue Good for newcomers

Comments

@rynowak
Copy link
Contributor

rynowak commented May 24, 2023

Bug information

Steps to reproduce (required)

Run the following:

rad bicep publish --file test/functional/corerp/resources/testdata/recipes/test-recipes/dapr-state-store.bicep --target br:rynowak.azurecr.io/test/functional/corerp/recipes/dapr-state-store

You can substitute any file, and any destination, but leave off the tag.

Observed behavior (required)

The error output:

Building test/functional/corerp/resources/testdata/recipes/test-recipes/dapr-state-store.bicep...
Error: invalid target "rynowak.azurecr.io/test/functional/corerp/recipes/dapr-state-store"

This error message isn't very helpful. It should explain the format, or what's wrong.

Desired behavior (required)

rad bicep publish targets the lack of a tag and infers latest

OR

The error message is improved.

Workaround (optional)

Specify the latest tag explicitly.

AB#7967

AB#9501

@rynowak
Copy link
Contributor Author

rynowak commented May 24, 2023

/cc @ytimocin

@ytimocin
Copy link
Contributor

This shouldn't be a blocker for this release, right? Can it be handled in the upcoming sprint and go with the next release? cc/ @rynowak @AaronCrawfis

@rynowak
Copy link
Contributor Author

rynowak commented May 24, 2023

Not a blocker, just minor polish.

@shalabhms shalabhms added the good first issue Good for newcomers label May 25, 2023
@willtsai willtsai transferred this issue from radius-project/radius Sep 19, 2023
@AaronCrawfis AaronCrawfis transferred this issue from another repository Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants