Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SAASINT-3244] DDS: Asana: Crawler Integration v1.0.0 #18997

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

tirthrajchaudhari-crest
Copy link
Contributor

What does this PR do?

This is a initial release PR of Asana integration including all the required assets.

Additional Notes

  • Crawler code for this integration has been committed in its respective repo
  • Pipeline and Facet group created for this integration are available in our sandbox and would be shared separately with the required teams.
  • OOTB detection rules JSON would be shared separately with the required teams as a part of separate repository.
  • Since during the standard attribute remapping we are not preserving the source attributes as per suggested best practices, it would result in filters using these standard attributes populating the values of other integrations as well as per current Datadog behaviour.

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

@tirthrajchaudhari-crest tirthrajchaudhari-crest marked this pull request as ready for review November 18, 2024 10:32
@hestonhoffman hestonhoffman added the editorial review Waiting on a more in-depth review from a docs team editor label Nov 18, 2024
@apiazza-dd apiazza-dd changed the title DDS: Asana: Crawler Integration v1.0.0 [SAASINT-2342] DDS: Asana: Crawler Integration v1.0.0 Nov 19, 2024
asana/README.md Outdated

This integration ingests the following logs:

- Audit Logs: Audit logs offer a record of important events, empowering your teams to detect and minimize the impact of incidents when they occur, and scrutinize configuration edits.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Audit Logs: Audit logs offer a record of important events, empowering your teams to detect and minimize the impact of incidents when they occur, and scrutinize configuration edits.
- **Audit Logs**: Audit logs provide a detailed record of significant events, enabling your teams to identify and mitigate the impact of incidents as they arise, while also reviewing configuration changes with precision.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

asana/README.md Outdated

- Audit Logs: Audit logs offer a record of important events, empowering your teams to detect and minimize the impact of incidents when they occur, and scrutinize configuration edits.

This integration collects audit logs and sends them to Datadog for analysis. Datadog uses the built-in logs pipeline to parse and enrich these logs, enabling effortless search and analysis. The integration provides insight into activities on the Asana platform through the out-of-the-box dashboards. Also, this integration provides out-of-the-box detection rules.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This integration collects audit logs and sends them to Datadog for analysis. Datadog uses the built-in logs pipeline to parse and enrich these logs, enabling effortless search and analysis. The integration provides insight into activities on the Asana platform through the out-of-the-box dashboards. Also, this integration provides out-of-the-box detection rules.
This integration gathers audit logs and forwards them to Datadog for seamless analysis. Datadog leverages its built-in log pipelines to parse and enrich these logs, facilitating easy search and detailed insights. With preconfigured dashboards, the integration offers clear visibility into activities within the Asana platform. Additionally, it includes ready-to-use detection rules for enhanced monitoring and security.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

asana/README.md Outdated

#### Get Credentials of Asana

1. Log in to your [Asana admin console][3] as **super admin** of an Enterprise+ organization.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
1. Log in to your [Asana admin console][3] as **super admin** of an Enterprise+ organization.
1. Log in to your [Asana Admin Console][3] as a **Super Admin** of an Enterprise+ organization.
2. Navigate to the **Apps** tab in your Admin Console.
3. Click on **Service Accounts**.
4. Select the **Add Service Account** button and complete the following steps:
1. Enter a descriptive and identifiable name under **Name**.
2. Under **Permission Scopes**, select **Scoped permissions** and check the **Audit Logs** box.
5. Click **Save Changes** and copy the **Service Account PAT** for later use.
6. Go to the **Settings** tab in your Admin Console.
7. Scroll to the bottom of the page to locate the **Domain ID (Workspace ID)**.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

asana/README.md Outdated
6. Navigate to the **Settings** tab from your admin console.
7. Scroll down till the end of the page, where you can find **Domain ID (Workspace ID)**.

#### Add your Asana Credentials
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
#### Add your Asana Credentials
#### Enter the following Asana credentials into the Datadog Asana Integration Tile.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think Enter the following Asana credentials into the Datadog Asana Integration Tile. should be a heading. Just a sentence under the Add your Asana Credentials heading.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@apiazza-dd Could you please check the attached screenshots of the Slack conversation with David regarding this change?
image
image

Copy link
Contributor

Choose a reason for hiding this comment

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

@tirthrajchaudhari-crest Sounds good, please defer to David & Doc's instructions. Thanks!

asana/README.md Outdated

### Logs

The Asana integration collects and forwards audit logs to Datadog.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add "For more details on the logs we collect with this integration, see the Asana Audit Logs API Docs4.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

"configuration": "README.md#Setup",
"support": "README.md#Support",
"changelog": "CHANGELOG.md",
"description": "Gain insights into Asana audit logs",
Copy link
Contributor

Choose a reason for hiding this comment

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

Explore and analyze Asana audit logs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

asana/README.md Outdated
6. Navigate to the **Settings** tab from your admin console.
7. Scroll down till the end of the page, where you can find **Domain ID (Workspace ID)**.

#### Add your Asana Credentials
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think Enter the following Asana credentials into the Datadog Asana Integration Tile. should be a heading. Just a sentence under the Add your Asana Credentials heading.

@torosmassa torosmassa changed the title [SAASINT-2342] DDS: Asana: Crawler Integration v1.0.0 [SAASINT-3244] DDS: Asana: Crawler Integration v1.0.0 Nov 19, 2024
Copy link

@jnhunsberger jnhunsberger left a comment

Choose a reason for hiding this comment

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

One small copy change is needed

asana/README.md Outdated

- **Audit Logs**: Audit logs provide a detailed record of significant events, enabling your teams to identify and mitigate the impact of incidents as they arise, while also reviewing configuration changes with precision.

This integration gathers audit logs and forwards them to Datadog for seamless analysis. Datadog leverages its built-in log pipelines to parse and enrich these logs, facilitating easy search and detailed insights. With preconfigured dashboards, the integration offers clear visibility into activities within the Asana platform. Additionally, it includes ready-to-use detection rules for enhanced monitoring and security.

Choose a reason for hiding this comment

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

Additionally, it includes ready-to-use Cloud SIEM detection rules for enhanced monitoring and security.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@BoyangHuang BoyangHuang added the assets/deploy-logs-staging ONLY USED BY Logs Backend - Validates that a PR is OK to go to staging label Jan 2, 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.

8 participants