Skip to content

Conversation

@dcsanabria
Copy link

Read skip_ssl_validation property from datadog.yaml and pass it through post request for verification as configured in the yaml

…gh post request for verification as configured in the yaml
@dcsanabria dcsanabria requested a review from a team as a code owner September 18, 2024 03:25
@dcsanabria
Copy link
Author

Hi @ian28223 here some changes for the skip_ssl_verification flag, tested in a local boxes and working as expected

@ian28223
Copy link
Contributor

Hi @ian28223 here some changes for the skip_ssl_verification flag, tested in a local boxes and working as expected

I'm not a member of the team responsible for reviewing code changes to this repository. Let's leave that to the Agent Delivery team.

@renantelo
Copy link

Hey @DataDog/agent-delivery could you please take a look here?

Copy link

@Pythyu Pythyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@ian28223
Copy link
Contributor

ian28223 commented Apr 1, 2025

/merge

@dd-devflow
Copy link

dd-devflow bot commented Apr 1, 2025

View all feedbacks in Devflow UI.
2025-04-01 23:42:08 UTC ℹ️ Start processing command /merge


2025-04-01 23:42:13 UTC ℹ️ 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.


2025-04-02 03:52:27 UTC ⚠️ MergeQueue: This merge request was unqueued

devflow unqueued this merge request: It did not become mergeable within the expected time

@ian28223
Copy link
Contributor

/merge

@dd-devflow
Copy link

dd-devflow bot commented Apr 16, 2025

View all feedbacks in Devflow UI.

2025-04-16 01:23:17 UTC ℹ️ Start processing command /merge


2025-04-16 01:23:22 UTC ℹ️ 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.


2025-04-16 05:34:10 UTC ⚠️ MergeQueue: This merge request was unqueued

devflow unqueued this merge request: It did not become mergeable within the expected time

@ian28223
Copy link
Contributor

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Sep 17, 2025

View all feedbacks in Devflow UI.

2025-09-17 05:47:28 UTC ℹ️ Start processing command /merge


2025-09-17 05:47:33 UTC ℹ️ 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.


2025-09-17 09:48:06 UTC ⚠️ MergeQueue: This merge request was unqueued

devflow unqueued this merge request: It did not become mergeable within the expected time

@ian28223
Copy link
Contributor

ian28223 commented Nov 4, 2025

Addressed in #149

@ian28223 ian28223 closed this Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants