opensource-observer / oso

Measuring the impact of open source software
https://opensource.observer
Apache License 2.0
66 stars 17 forks source link

bq2clickhouse is broken #1837

Closed ryscheng closed 1 month ago

ryscheng commented 1 month ago

Which area(s) are affected? (leave empty if unsure)

No response

To Reproduce

just run any of the jobs

Describe the Bug

logs show a gs://gs://bucket path, so there's so incorrect path construction happening

Expected Behavior

Works

ryscheng commented 1 month ago

Address the path issue here https://github.com/opensource-observer/oso/pull/1836

ryscheng commented 1 month ago

Address secrets management here https://github.com/opensource-observer/oso/pull/1835

ryscheng commented 1 month ago

Address path issue take 2 https://github.com/opensource-observer/oso/pull/1838

There was an issue with how we were deleting blobs

ryscheng commented 1 month ago

One more issue with triple slashes in the path https://github.com/opensource-observer/oso/pull/1839/files

ryscheng commented 1 month ago

Looks like we're really close. Clickhouse has all the marts now, but the jobs are still failing due to a permissions issue with deleting the old GCS blobs

ryscheng commented 1 month ago

On that note, we should get to

ryscheng commented 1 month ago

This is working with @ravenac95 's help fixing up the service account permissions!