Skip to content

Conversation

@leehinman
Copy link
Contributor

What does this PR do?

Removes duplicate ListGroupsForUser in aws/cloudtrail dataset

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all datasets collect metrics or logs.

@leehinman leehinman added the bug Something isn't working, use only for issues label Aug 25, 2020
@leehinman leehinman requested a review from kaiyan-sheng August 25, 2020 21:55
@elasticmachine
Copy link

elasticmachine commented Aug 25, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #256 updated]

  • Start Time: 2020-08-26T02:21:20.787+0000

  • Duration: 2 min 28 sec

@kaiyan-sheng
Copy link
Contributor

Tested this PR locally and this fixed error in elastic/package-storage#370

@kaiyan-sheng
Copy link
Contributor

@mtojek What's the process to update elastic/package-storage#370? Should we bump the version number here and just close elastic/package-storage#370?

- bump version to 0.2.8
@mtojek
Copy link
Contributor

mtojek commented Aug 26, 2020

@kaiyan-sheng

In this situation, the best flow would be as following:

  1. Close [snapshot] Update "aws" integration to version 0.2.7 package-storage#370
  2. Merge this PR
  3. Jenkins automatically opens a PR to package-storage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants