integrations-core icon indicating copy to clipboard operation
integrations-core copied to clipboard

[SIEMINT-122] DDSaaS: Brevo: Integration v1.0.0

Open rajshah-crest opened this issue 1 year ago • 1 comments

What does this PR do?

This is a initial release PR of Brevo integration including all the required assets i.e. Pipeline, Dashboard and Monitors

Additional Notes

  • This integration is webhook based and so Crawler will not be used.
  • Datadog team will be required to add UI component for generation of webhook URLs in the configuration tile. Two URLs will be created, with both URLs having the ddsource query parameter set to brevo. The service query parameter will be set to marketing-events for marketing events and transactional-events for transactional events.
  • The assets are available in our sandbox and can be shared separately with the required teams.

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e) [ ] Changelog entries must be created for modifications to shipped code [ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA. [ ] If you need to backport this PR to another branch, you can add the backport/ label to the PR and it will automatically open a backport PR once this one is merged

rajshah-crest avatar Sep 11 '24 10:09 rajshah-crest

For logs files, it is looking very good 👍, good job on this. I am pushing it in our test environment and will give you news soon for a potential approval.

thibaultkrebs avatar Sep 26 '24 13:09 thibaultkrebs

/merge

apiazza-dd avatar Nov 19 '24 13:11 apiazza-dd

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-11-19 13:58:50 UTC :information_source: MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals. Note: if you pushed new commits since the last approval, you may need additional approval. You can remove it from the waiting list with /remove command.


2024-11-19 14:02:28 UTC :warning: MergeQueue: This merge request was unqueued

This merge request was unqueued

dd-devflow[bot] avatar Nov 19 '24 13:11 dd-devflow[bot]

/merge --cancel

apiazza-dd avatar Nov 19 '24 14:11 apiazza-dd

Devflow running: /merge --cancel

View all feedbacks in Devflow UI.


2024-11-19 14:01:03 UTC :rotating_light: Devflow: /merge --cancel

Failed to generate input: unknown flag:

If you need support, contact us on Slack #devflow with those details!

dd-devflow[bot] avatar Nov 19 '24 14:11 dd-devflow[bot]

/merge -c

apiazza-dd avatar Nov 19 '24 14:11 apiazza-dd

/merge

apiazza-dd avatar Nov 19 '24 14:11 apiazza-dd

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-11-19 14:08:13 UTC :information_source: MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals. Note: if you pushed new commits since the last approval, you may need additional approval. You can remove it from the waiting list with /remove command.


2024-11-19 14:55:50 UTC :warning: MergeQueue: This merge request was unqueued

This merge request was unqueued

dd-devflow[bot] avatar Nov 19 '24 14:11 dd-devflow[bot]

/remove

apiazza-dd avatar Nov 19 '24 14:11 apiazza-dd

Devflow running: /remove

View all feedbacks in Devflow UI.


2024-11-19 14:55:48 UTC :information_source: Devflow: /remove

dd-devflow[bot] avatar Nov 19 '24 14:11 dd-devflow[bot]

@bparmar-crest @thibaultkrebs Can you please take another look? We've approved on our end but are blocked on merging

apiazza-dd avatar Nov 19 '24 14:11 apiazza-dd

@apiazza-dd Unfortunately, I cannot access the error links from above comments. Those links might have more info about merge failures. Maybe @thibaultkrebs can help here !

bparmar-crest avatar Nov 20 '24 05:11 bparmar-crest

@bparmar-crest @thibaultkrebs ISorry, we're waiting for another code owner review on our end for the pipeline. We'll let you know when it's merged.

apiazza-dd avatar Nov 21 '24 15:11 apiazza-dd

/merge

apiazza-dd avatar Dec 10 '24 23:12 apiazza-dd

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-10 23:00:31 UTC :information_source: MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals. Note: if you pushed new commits since the last approval, you may need additional approval. You can remove it from the waiting list with /remove command.


2024-12-11 03:00:34 UTC :warning: MergeQueue: This merge request was unqueued

This merge request was unqueued

dd-devflow[bot] avatar Dec 10 '24 23:12 dd-devflow[bot]

/merge

apiazza-dd avatar Dec 19 '24 20:12 apiazza-dd

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-19 20:17:00 UTC :information_source: MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals. Note: if you pushed new commits since the last approval, you may need additional approval. You can remove it from the waiting list with /remove command.


2024-12-19 21:29:37 UTC :warning: MergeQueue: This merge request was unqueued

[email protected] unqueued this merge request

dd-devflow[bot] avatar Dec 19 '24 20:12 dd-devflow[bot]

/merge -c

apiazza-dd avatar Dec 19 '24 21:12 apiazza-dd

/merge

apiazza-dd avatar Dec 19 '24 21:12 apiazza-dd

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-19 21:29:46 UTC :information_source: MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals. Note: if you pushed new commits since the last approval, you may need additional approval. You can remove it from the waiting list with /remove command.


2024-12-19 21:40:35 UTC :information_source: MergeQueue: This merge request was already merged

This pull request was merged directly.

dd-devflow[bot] avatar Dec 19 '24 21:12 dd-devflow[bot]