[SAASINT-3470] DDSaaS: Okta Workflows: Crawler Integration Without Assets #19281

DhruvaPatel-crest · 2024-12-18T11:52:23Z

What does this PR do?

PR includes skeleton of Okta Workflows integration without any assets like dashboards, image, README content, etc.

Motivation

This is beta release of crawler integration and is intended for internal testing before going live. As per @nathanmadams's suggestion, we are planning to raise separate PR with assets and all necessary information later.

Additional Notes

Only crawler-based integration related changes are included in this PR

Review checklist (to be filled by reviewers)

Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
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/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

rtrieu · 2024-12-18T20:27:58Z

okta_workflows/README.md

+## Overview
+[Okta Workflows][1] is a no-code automation platform provided by Okta, designed to simplify and automate identity-related tasks and processes. It allows organizations to build custom workflows that integrate seamlessly with Okta's identity and access management capabilities and third-party applications, enhancing operational efficiency, security, and user experience.
+
+The Okta Workflows integration collects okta workflow event logs and sends it into Datadog for comprehensive analysis.


Suggested change

The Okta Workflows integration collects okta workflow event logs and sends it into Datadog for comprehensive analysis.

The Okta Workflows integration collects Okta workflow event logs and sends it into Datadog for comprehensive analysis.

Done changes as per suggestion.

rtrieu · 2024-12-18T20:28:28Z

Created DOCS-9776 for docs review.

drichards-87 · 2024-12-19T21:56:23Z

Hi @DhruvaPatel-crest. I'll be handling the Docs Team review, so feel free to reach out when the PR is ready to reviewed. Thanks!

DhruvaPatel-crest · 2024-12-26T05:59:17Z

Hi @drichards-87 PR is ready for review.

DhruvaPatel-crest · 2024-12-26T06:00:06Z

Hi @drichards-87 PR is ready for review.

drichards-87 · 2024-12-30T16:21:50Z

Hi @DhruvaPatel-crest. The Setup, Configuration, and Validation sections still have placeholder content. Is that intentional?

drichards-87

Left a couple of suggestions from Docs.

drichards-87 · 2024-12-30T19:55:13Z

okta_workflows/README.md

+## Overview
+[Okta Workflows][1] is a no-code automation platform provided by Okta, designed to simplify and automate identity-related tasks and processes. It allows organizations to build custom workflows that integrate seamlessly with Okta's identity and access management capabilities and third-party applications, enhancing operational efficiency, security, and user experience.
+
+The Okta Workflows integration collects Okta workflow event logs and sends it into Datadog for comprehensive analysis.


Suggested change

The Okta Workflows integration collects Okta workflow event logs and sends it into Datadog for comprehensive analysis.

The Okta Workflows integration collects Okta workflow event logs and sends them to Datadog for comprehensive analysis.

Done changes as per suggestion.

drichards-87 · 2024-12-30T19:55:25Z

okta_workflows/README.md

+
+### Logs
+
+The Okta Workflows integration collects and forwards okta workflow event logs to Datadog.


Suggested change

The Okta Workflows integration collects and forwards okta workflow event logs to Datadog.

The Okta Workflows integration collects and forwards Okta workflow event logs to Datadog.

Done changes as per suggestion.

DhruvaPatel-crest · 2024-12-31T11:33:02Z

Hi @DhruvaPatel-crest. The Setup, Configuration, and Validation sections still have placeholder content. Is that intentional?

Yes, this is intentional. Without assets PR, it contains the initial structure for crawler based integration. Once this PR is merged, we will raise a separate PR with the full assets.

drichards-87 · 2024-12-31T16:14:44Z

Hi @DhruvaPatel-crest. The Setup, Configuration, and Validation sections still have placeholder content. Is that intentional?

Yes, this is intentional. Without assets PR, it contains the initial structure for crawler based integration. Once this PR is merged, we will raise a separate PR with the full assets.

Understood. Thanks for the clarification.

codecov · 2024-12-31T16:15:11Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.19%. Comparing base (724567b) to head (361179e).
Report is 165 commits behind head on master.

Additional details and impacted files

Flag	Coverage Δ
activemq	`?`
cassandra	`?`
hive	`?`
hivemq	`?`
hudi	`?`
ignite	`?`
jboss_wildfly	`?`
kafka	`?`
presto	`?`
solr	`?`

Flags with carried forward coverage won't be shown. Click here to find out more.

BoyangHuang · 2025-01-28T17:00:11Z

/merge

dd-devflow · 2025-01-28T17:01:03Z

Devflow running: `/merge`

View all feedbacks in Devflow UI.

2025-01-28 17:01:03 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-01-28 21:01:06 UTC ⚠️ MergeQueue: This merge request was unqueued

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

abhi-modugula · 2025-01-28T20:21:24Z

/merge

dd-devflow · 2025-01-28T20:21:32Z

Devflow running: `/merge`

View all feedbacks in Devflow UI.

2025-01-28 20:21:31 UTC ❌ MergeQueue

PR already in the queue with status waiting

abhi-modugula · 2025-01-28T20:36:41Z

/merge

dd-devflow · 2025-01-28T20:36:50Z

Devflow running: `/merge`

View all feedbacks in Devflow UI.

2025-01-28 20:36:49 UTC ❌ MergeQueue

PR already in the queue with status waiting

…sets (#19281) * initial commit * updated readme as per review suggestions * updated readme file as per PR comments * turn on public docs * turn off public docs --------- Co-authored-by: abhi-modugula <[email protected]> 3287d5a

initial commit

63ba65f

datadog-assets bot added docs/review-requested ecosystems/review-requested product/review-requested labels Dec 18, 2024

DhruvaPatel-crest changed the title ~~DDSaaS: Okta Workflows: Crawler Integration ithout Assets~~ DDSaaS: Okta Workflows: Crawler Integration Without Assets Dec 18, 2024

DhruvaPatel-crest marked this pull request as ready for review December 18, 2024 12:06

DhruvaPatel-crest requested review from a team as code owners December 18, 2024 12:06

DhruvaPatel-crest changed the title ~~DDSaaS: Okta Workflows: Crawler Integration Without Assets~~ [SAASINT-3470] DDSaaS: Okta Workflows: Crawler Integration Without Assets Dec 18, 2024

rtrieu reviewed Dec 18, 2024

View reviewed changes

rtrieu added the editorial review Waiting on a more in-depth review from a docs team editor label Dec 18, 2024

drichards-87 requested a review from a team December 19, 2024 18:30

updated readme as per review suggestions

6a4a204

DhruvaPatel-crest requested a review from rtrieu December 20, 2024 08:47

rtrieu removed their request for review December 23, 2024 17:08

HadhemiDD previously approved these changes Dec 30, 2024

View reviewed changes

datadog-assets bot added the agent/approved label Dec 30, 2024

drichards-87 requested changes Dec 30, 2024

View reviewed changes

datadog-assets bot added docs/requested-changes and removed docs/review-requested labels Dec 30, 2024

updated readme file as per PR comments

361179e

DhruvaPatel-crest dismissed HadhemiDD’s stale review via 361179e December 31, 2024 10:26

drichards-87 previously approved these changes Dec 31, 2024

View reviewed changes

datadog-assets bot added docs/approved and removed docs/requested-changes labels Dec 31, 2024

BoyangHuang previously approved these changes Jan 28, 2025

View reviewed changes

temporal-github-worker-1 bot added agent/review-requested and removed agent/approved labels Jan 28, 2025

dd-devflow bot added the mergequeue-status: waiting label Jan 28, 2025

Merge branch 'master' into okta_workflows-v1.0.0

f465bec

dd-devflow bot added mergequeue-status: removed and removed mergequeue-status: waiting labels Jan 28, 2025

turn on public docs

373d135

abhi-modugula dismissed stale reviews from BoyangHuang and drichards-87 via 373d135 January 29, 2025 15:12

temporal-github-worker-1 bot added docs/review-requested and removed docs/approved labels Jan 29, 2025

turn off public docs

694ca9a

abhi-modugula approved these changes Jan 29, 2025

View reviewed changes

obi11235 approved these changes Jan 29, 2025

View reviewed changes

temporal-github-worker-1 bot added agent/approved and removed agent/review-requested labels Jan 29, 2025

abhi-modugula added this pull request to the merge queue Jan 29, 2025

Merged via the queue into DataDog:master with commit 3287d5a Jan 29, 2025
36 of 37 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[SAASINT-3470] DDSaaS: Okta Workflows: Crawler Integration Without Assets #19281

[SAASINT-3470] DDSaaS: Okta Workflows: Crawler Integration Without Assets #19281

DhruvaPatel-crest commented Dec 18, 2024

rtrieu Dec 18, 2024

DhruvaPatel-crest Dec 20, 2024

rtrieu commented Dec 18, 2024

drichards-87 commented Dec 19, 2024

DhruvaPatel-crest commented Dec 26, 2024

DhruvaPatel-crest commented Dec 26, 2024

drichards-87 commented Dec 30, 2024

drichards-87 left a comment

drichards-87 Dec 30, 2024

DhruvaPatel-crest Dec 31, 2024

drichards-87 Dec 30, 2024

DhruvaPatel-crest Dec 31, 2024

DhruvaPatel-crest commented Dec 31, 2024

drichards-87 commented Dec 31, 2024

codecov bot commented Dec 31, 2024 •

edited

Loading

BoyangHuang commented Jan 28, 2025

dd-devflow bot commented Jan 28, 2025 •

edited

Loading

abhi-modugula commented Jan 28, 2025

dd-devflow bot commented Jan 28, 2025 •

edited

Loading

abhi-modugula commented Jan 28, 2025

dd-devflow bot commented Jan 28, 2025 •

edited

Loading

	The Okta Workflows integration collects okta workflow event logs and sends it into Datadog for comprehensive analysis.
	The Okta Workflows integration collects Okta workflow event logs and sends it into Datadog for comprehensive analysis.


		### Logs

		The Okta Workflows integration collects and forwards okta workflow event logs to Datadog.

[SAASINT-3470] DDSaaS: Okta Workflows: Crawler Integration Without Assets #19281

[SAASINT-3470] DDSaaS: Okta Workflows: Crawler Integration Without Assets #19281

Conversation

DhruvaPatel-crest commented Dec 18, 2024

What does this PR do?

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

rtrieu Dec 18, 2024

Choose a reason for hiding this comment

DhruvaPatel-crest Dec 20, 2024

Choose a reason for hiding this comment

rtrieu commented Dec 18, 2024

drichards-87 commented Dec 19, 2024

DhruvaPatel-crest commented Dec 26, 2024

DhruvaPatel-crest commented Dec 26, 2024

drichards-87 commented Dec 30, 2024

drichards-87 left a comment

Choose a reason for hiding this comment

drichards-87 Dec 30, 2024

Choose a reason for hiding this comment

DhruvaPatel-crest Dec 31, 2024

Choose a reason for hiding this comment

drichards-87 Dec 30, 2024

Choose a reason for hiding this comment

DhruvaPatel-crest Dec 31, 2024

Choose a reason for hiding this comment

DhruvaPatel-crest commented Dec 31, 2024

drichards-87 commented Dec 31, 2024

codecov bot commented Dec 31, 2024 • edited Loading

Codecov Report

BoyangHuang commented Jan 28, 2025

dd-devflow bot commented Jan 28, 2025 • edited Loading

Devflow running: /merge

abhi-modugula commented Jan 28, 2025

dd-devflow bot commented Jan 28, 2025 • edited Loading

Devflow running: /merge

abhi-modugula commented Jan 28, 2025

dd-devflow bot commented Jan 28, 2025 • edited Loading

Devflow running: /merge

codecov bot commented Dec 31, 2024 •

edited

Loading

dd-devflow bot commented Jan 28, 2025 •

edited

Loading

Devflow running: `/merge`

dd-devflow bot commented Jan 28, 2025 •

edited

Loading

Devflow running: `/merge`

dd-devflow bot commented Jan 28, 2025 •

edited

Loading

Devflow running: `/merge`