-
Notifications
You must be signed in to change notification settings - Fork 11
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
Download URL is wrong #28
Comments
This is probably a duplicate of #24 , an update has been pushed already by Chocomilk, but the automated review keeps failing. @sbaerlocher has already triggered Chocolatey's automatic verification many times, but it seems it always fails for an unclear reason (maybe disk space)? Sp the update is already there, but Chocolatey does not appear to be accepting it... |
@sbaerlocher Maybe requesting a verification exemption (as discussed e.g. here: chocolatey/choco#2520 ) would be best? |
@olifre Thank you for providing a more detailed analysis. I retested the issue using the Local Vagrant Test Environment available at https://github.com/chocolatey-community/chocolatey-test-environment from Chocolatey, and was able to confirm that it works in this environment. I will inquire on the Chocolatey Discord channel to see if they can make an exception or provide further details. |
download url changed again: today, April 11, 2023, a new version of the ODT, Version 16.0.16227.20258, was released by Microsoft -- see https://learn.microsoft.com/en-us/officeupdates/odt-release-history New download url is https://download.microsoft.com/download/2/7/A/27AF1BE6-DD20-4CB4-B154-EBAB8A7D4A7E/officedeploymenttool_16227-20258.exe |
@deargle Chocomilk has automatically updated to the new URL, it does check for new installers once every 24 hours: Interestingly, this version was finally approved by the Chocolatey repo maintainers. @sbaerlocher , do you understand why it suddenly "just works" again? |
Great! No, I'm not sure why this one got approved. |
link to ODT has changed again: do you think it is also possible to add the ODT binary itself direct into the choco package instead of update the url all time I know it will be more size (from 8KiB to 3.5 MiB) but as long as there are no permalinks to ODT it will be more stable |
Chocomilk has already updated to that: It will of course take a bit of time until it shows up in the Chocolatey repos, providing that this time, the packages don't time out during tests.
I am quite sure Microsoft does not allow to mirror / distribute the binaries. |
of course ... you are right: https://www.microsoft.com/en-us/legal/terms-of-use ... no way to add a 100% stable solution here. |
It seems the dreaded timeout has caused the package tests to fail once more 😢 . |
current failed validation come from #29 |
Validation didn't fail, this is just a recommendation, see the green checkmark on: Verification failed, though, the logs contain:
|
got it
but now it is: |
I have install that version now many time: and it was never blocked on my side like in that choco auto review process: Can you trigger the "Verification Testing" of "https://community.chocolatey.org/packages/microsoft-office-deployment/16.0.16501.20196" once again? |
@sbaerlocher Maybe you can take a look at triggering verification testing once more? Or maybe @pauby can take a look at adding some exception? While Chocomilk seems to reliably bump the version of this package, verification tests appear to run into timeouts (likely the testing environment is "too slow" for MS Office installation testing in the time limit). |
I have re-run the verification. If you need this re-run then adding a moderation comment on the package page is the best way to request it. |
@pauby Many thanks! Since @ThomasPatzig and me are just "users" and not maintainers of the package, I am not sure we can add moderation comments (and the maintainers don't seem to be that responsive at the time being). |
That version has now passed verification and has been approved! |
@pauby Many thanks! @CodeMonk3y84 If things also work for you, please close this issue. Chocomilk should update automatically as usually, if verification tests time out again the next time, we can open a new issue. |
Thank you for your work. |
@DmitriiKh Please don't 0-day-bump: @sbaerlocher has set up Chocomilk to run automatically once every day:
It did run 8 hours before your comment and did not find the new version available yet. So we should check back tomorrow, and then can check whether the new version is found correctly, uploaded and approved automatically. |
@olifre Thank you for your answer. I have just figured out why your CI/CD didn't pick the new version of ODT that was deployed yesterday. I took some time. Sorry about that. Small suggestion: could you please add this information to README please? Like ODT links are getting updated automatically during a couple of days. |
@DmitriiKh Sadly, I am also just a user of this package (like you), so I don't have any permissions to change things (otherwise, I would also have just re-triggered the CI pipeline after your comment). @sbaerlocher is maintaining this project, of course you could open a PR he might merge? |
ah... now I got it the chocomilk.yml observes via https://release-monitoring.org/project/21215/ for new version of ODT latest version update there was seen 2023-08-01 16:33 UTC -> 16.0.16626.20148 but last run of chocomilk.yml was 2023-08-01 15:16 GMT ~13:16 UTC and this had seen next run of chocomilk.yml will be ~ 2023-08-01 15:16 GMT ~13:16 UTC .... |
@ThomasPatzig Exactly ;-). Chocomilk did its commit here: Sadly, automatic verification failed once more, with the usual cause (timeout), see: @sbaerlocher has retriggered verification testing quite often in the past, and also done local testing (which always worked and took a few minutes). Sadly, it seems the automatic verification system is significantly slower and quite often runs into these timeouts. Maybe @pauby can help out again? The last time he triggered it in this very issue, this helped and the package stayed below the enforced timeout. |
@pauby Many thanks, once more, it worked fine on the second attempt / when you triggered it 😉 . Indeed, maybe it's time to go through the triaging process. Since the last "human" commit is ~2 months ago, maybe it's worth to wait for another upstream release. If that has issues again, it's probably time to go ahead with the triaging... |
Please update the ODT download link.
The text was updated successfully, but these errors were encountered: