Skip to content

Conversation

@ashb
Copy link
Member

@ashb ashb commented Jan 10, 2023

At least not on any URL I could guess at.

Removing this should un-break main.

@ashb
Copy link
Member Author

ashb commented Jan 10, 2023

Filed googleapis/google-cloud-python#10814 upstream

Copy link
Contributor

@Taragolis Taragolis left a 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:

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

@ashb
Copy link
Member Author

ashb commented Jan 10, 2023

Google are working on fixing this so we likely don't have to merge this PR

@potiuk
Copy link
Member

potiuk commented Jan 10, 2023

It's almost fixed while we wait for Google - but we need to add spellcheck exclusions:

SynthesizeSpeechResponse
TransferRun
StartManualTransferRunsResponse
TransferConfig
ListDatasetsPager
ListTableSpecsPager
ListColumnSpecsPager

Copy link
Member

@potiuk potiuk left a 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)

@potiuk
Copy link
Member

potiuk commented Jan 10, 2023

Google are working on fixing this so we likely don't have to merge this PR

I'd merge it anyway - we never know how long it will take them and whether it will be working at first try

@potiuk
Copy link
Member

potiuk commented Jan 10, 2023

We can always simply revert it

@potiuk potiuk merged commit 44d93be into apache:main Jan 10, 2023
@potiuk
Copy link
Member

potiuk commented Jan 10, 2023

Merge it - it was green.

@ashb ashb deleted the google-no-objects.inv-anymore branch January 10, 2023 20:48
ephraimbuddy pushed a commit that referenced this pull request Jan 12, 2023
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Jan 13, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.5.1 milestone Jan 13, 2023
potiuk added a commit to potiuk/airflow that referenced this pull request Feb 14, 2023
…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
potiuk added a commit that referenced this pull request Feb 14, 2023
…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
pierrejeambrun pushed a commit that referenced this pull request Mar 7, 2023
…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)
pierrejeambrun pushed a commit that referenced this pull request Mar 8, 2023
…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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) kind:documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants