-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Google have re-worked their docs site ane appear not publish objects.inv #28834
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
Conversation
At least not on any URL I could guess at.
|
Filed googleapis/google-cloud-python#10814 upstream |
Taragolis
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I add locally additional simple validation for inventories (will create as follow up PR).
The only two inventory related to google provider exists:
google-api-core: https://googleapis.dev/python/google-api-core/latest/objects.invgoogle-cloud-translate: https://googleapis.dev/python/translation/latest/objects.inv
I'm not sure should we keep them or not
#################### Fetching inventories ####################
To download 116 inventorie(s)
google-cloud-bigquery-datatransfer: Unexpected inventory header: <!doctype html>
google-cloud-automl: Unexpected inventory header: <!doctype html>
google-cloud-bigquery: Unexpected inventory header: <!doctype html>
google-cloud-bigtable: Unexpected inventory header: <!doctype html>
google-cloud-container: Unexpected inventory header: <!doctype html>
google-cloud-core: Unexpected inventory header: <!doctype html>
google-cloud-bigquery-storage: Unexpected inventory header: <!doctype html>
google-cloud-monitoring: Unexpected inventory header: <!doctype html>
google-cloud-datacatalog: Unexpected inventory header: <!doctype html>
google-cloud-kms: Unexpected inventory header: <!doctype html>
google-cloud-dlp: Unexpected inventory header: <!doctype html>
google-cloud-spanner: Unexpected inventory header: <!doctype html>
google-cloud-datastore: Unexpected inventory header: <!doctype html>
google-cloud-pubsub: Unexpected inventory header: <!doctype html>
google-cloud-language: Unexpected inventory header: <!doctype html>
google-cloud-redis: Unexpected inventory header: <!doctype html>
google-cloud-speech: Unexpected inventory header: <!doctype html>
google-cloud-storage: Unexpected inventory header: <!doctype html>
google-cloud-tasks: Unexpected inventory header: <!doctype html>
google-cloud-texttospeech: Unexpected inventory header: <!doctype html>
google-cloud-vision: Unexpected inventory header: <!doctype html>
google-cloud-videointelligence: Unexpected inventory header: <!doctype html>
Result: 94 success, 22 failed
Failed packages:
1. google-cloud-automl
2. google-cloud-bigquery
3. google-cloud-bigquery-datatransfer
4. google-cloud-bigquery-storage
5. google-cloud-bigtable
6. google-cloud-container
7. google-cloud-core
8. google-cloud-datacatalog
9. google-cloud-datastore
10. google-cloud-dlp
11. google-cloud-kms
12. google-cloud-language
13. google-cloud-monitoring
14. google-cloud-pubsub
15. google-cloud-redis
16. google-cloud-spanner
17. google-cloud-speech
18. google-cloud-storage
19. google-cloud-tasks
20. google-cloud-texttospeech
21. google-cloud-videointelligence
22. google-cloud-vision
|
Google are working on fixing this so we likely don't have to merge this PR |
|
It's almost fixed while we wait for Google - but we need to add spellcheck exclusions: |
potiuk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pending spellcheck adding. I think it will take a while for Google to bring the inventories back (even if they treat it as p1)
I'd merge it anyway - we never know how long it will take them and whether it will be working at first try |
|
We can always simply revert it |
|
Merge it - it was green. |
…objects.inv (apache#28834)" This reverts commit 44d93be. Google re-published the .inv files after deleting them in January. Details in googleapis/google-cloud-python#10814
…objects.inv (#28834)" (#29543) This reverts commit 44d93be. Google re-published the .inv files after deleting them in January. Details in googleapis/google-cloud-python#10814
…objects.inv (#28834)" (#29543) This reverts commit 44d93be. Google re-published the .inv files after deleting them in January. Details in googleapis/google-cloud-python#10814 (cherry picked from commit c2c27c3)
…objects.inv (#28834)" (#29543) This reverts commit 44d93be. Google re-published the .inv files after deleting them in January. Details in googleapis/google-cloud-python#10814 (cherry picked from commit c2c27c3)
At least not on any URL I could guess at.
Removing this should un-break main.