-
Notifications
You must be signed in to change notification settings - Fork 8
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
If version is not specified installation fails #9
Comments
I get this sometimes when installing deb packages by file path using dpkg. Usually |
@fabiosantoscode I got this error 1 out of 20 times, I was hoping to be differences in the GH image. But its also very hard to fix it because it's very hard to repro it. |
I've configured a |
Error# apt install ··· ./dvc.deb
E: Invalid archive signature
E: Internal error, could not locate member control.tar.{zstlz4gzxzbz2lzma}
E: Could not read meta data from /home/runner/work/dvc-action-test/dvc-action-test/dvc.deb
E: The package lists or status file could not be parsed or opened. Timeline
CauseThis race condition will happen in a time window of approximately 10 minutes after each DVC release, because published releases appear as Lines 34 to 40 in 19880d0
This issue was specially hard to narrow down because the Lines 20 to 32 in 19880d0
After reviewing the code, I'm inclined to think that there would not be any difference between specifying Line 44 in 19880d0
Solutions
|
Awesome catch!
GITHUB_TOKEN is accesible in the workflow. Is that GITHUB_TOKEN that you are referring to? |
@DavidGOrtega, yes, it's accessible, but it doesn't look appropriate to ask for it when setting up a workflow dependency; at least, not without an excellent reason. I would prefer either (1) to get DVC to publish releases once assets have been uploaded, or (2) to install the package from other official sources as @efiop suggested here. Nevertheless, it looks like our |
You mean this? - uses: iterative/setup-cml@v1
with:
token: ${{ secrets.GITHUB_TOKEN }} |
Yes, this would allow us to issue authenticated API calls and raise the rate limit, but at the cost of asking the user for a token on a step that (ideally) should not require it. Another possible option would be to hardcode the latest version on this repository and update it automatically with a |
Definitely, the ideal solution would be having all the DVC release assets uploaded before publishing. All the alternatives I can think of would add up to the technical debt laundry list, one way or another. 📖 Note: this issue reduces our layman availability metric to 99.88 %Considering that the release_dates = [
1614942525,
1614900985,
1614785918,
1614766914,
1614715882,
1614615657,
1614106844,
1614339349,
1613533904,
1613415465,
1612260018,
1612208847,
1611628591,
1611527735,
1611227359,
1609850097,
1608723473,
1608124083,
1608054971,
1607970359,
1607819710,
1607443941,
1606930143,
1606926602,
1606924068,
1606235653,
1605813745,
1605662310,
1603812829,
1603387362
]
release_intervals = [
abs(first - second)
for first, second in
zip(releases, releases[1:])
]
average_release_interval = sum(release_intervals) / len(release_intervals)
estimated_downtime_ratio = 8 * 60 / average_release_interval
print(f"Estimated availability {1 - estimated_downtime_ratio:.2%}") |
Version should be
latest
by default. However this worksand this returns an error
The text was updated successfully, but these errors were encountered: