Skip to content

CSOAR-3364: Added IAM related change logs in AWS integrations #5636

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Aug 21, 2025
Merged

CSOAR-3364: Added IAM related change logs in AWS integrations #5636

merged 5 commits into from
Aug 21, 2025

Conversation

@ruturajsumo
Copy link
Collaborator

@ruturajsumo ruturajsumo commented Jul 29, 2025

Purpose of this pull request

This pull request...

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/CSOAR-3364

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Jul 29, 2025
@ruturajsumo ruturajsumo self-assigned this Jul 29, 2025
@ruturajsumo ruturajsumo marked this pull request as draft July 29, 2025 07:27
@jpipkin1 jpipkin1 marked this pull request as ready for review August 21, 2025 14:33
@jpipkin1
Copy link
Collaborator

jpipkin1 commented Aug 21, 2025 •
edited
Loading

@ruturajsumo, Shubham Chauhan asked that I review and publish this PR (see his comment in CSOAR-3697). So I changed the PR status to "Ready to Review", and I will go ahead and review and publish the PR.

@jpipkin1
Copy link
Collaborator

jpipkin1 commented Aug 21, 2025 •
edited
Loading

I'm placing this PR back into draft mode. Its build errors can't be resolved until PR #5630 is merged.

@jpipkin1 jpipkin1 marked this pull request as draft August 21, 2025 14:57
@jpipkin1 jpipkin1 marked this pull request as ready for review August 21, 2025 19:35
jpipkin1
jpipkin1 approved these changes Aug 21, 2025
View reviewed changes
Copy link
Collaborator

@jpipkin1 jpipkin1 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I placed the PR back into "Ready to Review" status.

@jpipkin1
Copy link
Collaborator

jpipkin1 commented Aug 21, 2025

Approved, @ruturajsumo. I made small updates (see commit "Updates from review").

@jpipkin1 jpipkin1 added this pull request to the merge queue Aug 21, 2025
Merged via the queue into SumoLogic:main with commit 82329b8 Aug 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jpipkin1 jpipkin1 jpipkin1 approved these changes

@kimsauce kimsauce Awaiting requested review from kimsauce kimsauce is a code owner

@JV0812 JV0812 Awaiting requested review from JV0812 JV0812 is a code owner

@mafsumo mafsumo Awaiting requested review from mafsumo mafsumo is a code owner

@amee-sumo amee-sumo Awaiting requested review from amee-sumo amee-sumo is a code owner

+1 more reviewer

@nileshgaikwad-sumo nileshgaikwad-sumo nileshgaikwad-sumo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ruturajsumo ruturajsumo

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ruturajsumo @jpipkin1 @nileshgaikwad-sumo