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

Auto Update Version is broken #3384

Closed
svrnm opened this issue Oct 16, 2023 · 3 comments · Fixed by #3385
Closed

Auto Update Version is broken #3384

svrnm opened this issue Oct 16, 2023 · 3 comments · Fixed by #3385
Assignees
Labels
bug Something isn't working CI/infra CI & infrastructure p0-critical

Comments

@svrnm svrnm added bug Something isn't working CI/infra CI & infrastructure p0-critical labels Oct 16, 2023
@chalin chalin self-assigned this Oct 16, 2023
@chalin
Copy link
Contributor

chalin commented Oct 16, 2023

I can reproduce the problem locally:

$ .github/workflows/scripts/auto-update-version.sh opentelemetry-specification spec scripts/content-modules/adjust-pages.pl
...
+ git fetch
From https://github.com/open-telemetry/opentelemetry-specification
 * [new tag]         v1.25.0    -> v1.25.0
+ git switch --detach v1.26.0
fatal: invalid reference: v1.26.0

We've faced a similar issue before (#3103). The fix is the same too: when setting DEPTH appropriately (see #3385), the command shown above works.

@chalin
Copy link
Contributor

chalin commented Oct 16, 2023

@chalin
Copy link
Contributor

chalin commented Oct 16, 2023

The script worked, but the individual PRs are failing. I suspect that it's because they are mutually dependent on each other. 👀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI/infra CI & infrastructure p0-critical
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants