airbytehq / airbyte

The leading data integration platform for ETL / ELT data pipelines from APIs, databases & files to data warehouses, data lakes & data lakehouses. Both self-hosted and Cloud-hosted.
https://airbyte.com
Other
16.17k stars 4.13k forks source link

🤖 /publish command: auto bumping version for multiple connectors in same PR fails #12276

Closed Phlair closed 2 years ago

Phlair commented 2 years ago

Tell us about the problem you're trying to solve

If you /publish multiple connectors at the same in the same PR, the auto-bump will likely fail on any after the first because the git HEAD isn't up-to-date with the newly auto pushed changes from the first publish.

See example here: https://github.com/airbytehq/airbyte/pull/12274

Describe the solution you’d like

Add a fetch + pull before we do the push from Octavia so we can catch this most of the time (there is still a window that this could fail on but it's significantly decreased with this fix)

sherifnada commented 2 years ago

@Phlair can you add this to the backlog grooming agenda?