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

--ignore-missing doesn't work #153

Closed
feinstein opened this issue Sep 5, 2023 · 11 comments · Fixed by #168
Closed

--ignore-missing doesn't work #153

feinstein opened this issue Sep 5, 2023 · 11 comments · Fixed by #168
Assignees

Comments

@feinstein
Copy link

I got an error at the end of the upload saying:

error: Could not find the SHA of the previous release in the git history. If you limit the clone depth, try to increase it. Otherwise, it means that the commit we are looking for was amended or squashed and cannot be retrieved. Use --ignore-missing flag to skip it and create a new release with the default commits count.

Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.
Please attach the full debug log to all bug reports.

exitCode: 1

So then I added the --ignore-missing flag and run dart run sentry_dart_plugin --ignore-missing, but then I continue to get the same error.

The commit was squashed.

@buenaflor
Copy link
Contributor

Hi! thank you for the issue, I'll be looking into it

@buenaflor buenaflor self-assigned this Sep 6, 2023
@feinstein
Copy link
Author

Just a little bit more info, I develop new features into a branch and then I squash and merge this branch into the development branch. So I think I uploaded the debug symbols in a feature branch (the feature was to add sentry and test sentry debug symbols upload), and later when I merge/squased this branch I can't upload the symbols without seeing an error.

@buenaflor buenaflor assigned denrase and unassigned buenaflor Sep 12, 2023
@mattsolle
Copy link

Any update? This is effecting our application's CI/CD pipeline for production.

@brightknight08
Copy link

happening to my projects too.

@denrase
Copy link
Collaborator

denrase commented Oct 2, 2023

We are looking into this now, thank you for reporting this.

@kahest kahest moved this from Needs Discussion to Needs Investigation in Mobile & Cross Platform SDK Oct 2, 2023
@denrase
Copy link
Collaborator

denrase commented Oct 2, 2023

@feinstein Just a question, does it work as expected when you add ignore_missing: true to the yaml config?

ignore_missing: true

@denrase
Copy link
Collaborator

denrase commented Oct 2, 2023

This is probably an error message from sentry-cli, which we are using/wrapping. Everything in the dart plugin should be configured via the yaml config and not by command line parameters.

@feinstein
Copy link
Author

It does work when I add it to the yaml file.

This limitation would be useful to be added into the docs.

@denrase
Copy link
Collaborator

denrase commented Oct 3, 2023

@feinstein Yes, It's not ideal that the error references a solution that does not work for the plugin, but rather for the underlying component we are using. Only adds confusion.

@mattsolle @brightknight08 Please also check if adding ignore_missing: true to your configuration to see if this resolves your issue.

@denrase
Copy link
Collaborator

denrase commented Oct 9, 2023

@feinstein We have updated our troubleshooting section.

@feinstein
Copy link
Author

Great, thanks a lot!

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

Successfully merging a pull request may close this issue.

5 participants