airbyte icon indicating copy to clipboard operation
airbyte copied to clipboard

✨ Source Gitlab: add new streams DORA metrics & Group Issues

Open CyprienBarbault opened this issue 1 year ago • 3 comments

What

This PR is adding 5 new streams :

  • Group Issues (https://docs.gitlab.com/ee/api/issues.html#list-group-issues) And the 4 DORA metrics :
  • Change_failure_rate
  • Deployment_frequency
  • Lead_time_for_changes
  • Time_to_restore_service

How

Review guide

User Impact

Can this PR be safely reverted and rolled back?

  • [ ] YES 💚
  • [ ] NO ❌

CyprienBarbault avatar May 16 '24 15:05 CyprienBarbault

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 28, 2024 8:03am

vercel[bot] avatar May 16 '24 15:05 vercel[bot]

Linked issue : https://github.com/airbytehq/airbyte/issues/38291

CyprienBarbault avatar May 16 '24 15:05 CyprienBarbault

I'm not sure if it should be a major or a minor version bump as there is changes to the api_url

CyprienBarbault avatar May 16 '24 15:05 CyprienBarbault

@CyprienBarbault were you able to check my last comment? Do you want to continue this contribution?

marcosmarxm avatar Jun 13 '24 11:06 marcosmarxm

@CyprienBarbault were you able to check my last comment? Do you want to continue this contribution?

Hello @marcosmarxm I was not able to work on this. I do want to have this work merged to the Airbyte repo, so either you could continue my work, or I will go back to it once I have some time

CyprienBarbault avatar Jun 14 '24 13:06 CyprienBarbault

@marcosmarxm I made the requested changes, but I'm unable to run the airbyte-ci. Waiting on your input

CyprienBarbault avatar Jun 20 '24 14:06 CyprienBarbault

Hey @marcosmarxm my company has stopped using Airbyte to pull DORA metrics because of the lack of support. We found an other alternative. I won't have more time to work on this, but you can continue the MR.

CyprienBarbault avatar Jul 05 '24 13:07 CyprienBarbault

@CyprienBarbault sorry to hear that. Unfortunately we don't have bandwidth to get this merged now.

marcosmarxm avatar Jul 05 '24 13:07 marcosmarxm