Skip to content

Feature/add ecs #182

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
Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open

Feature/add ecs #182

wants to merge 33 commits into from

Conversation

@daniebrill
Copy link

What type of PR is this?
Backend Feature

What this PR does / why we need it:
Add checks for ECS

Does this PR introduce a breaking change, if so what is it?:

NONE

Additional documentation:

Jackafive753 and others added 30 commits May 28, 2021 14:10
@Jackafive753
#SOS-15
8daf59a 
/collector/structs.go
+ AccountSpecifiedFiels struct to save AccountId and AccountName

/collector/aws/common/structs.go
+ add Method getAccountName()

/collector/aws/detector.go
+ save in DetectorManager the accountName from config file

/collector/resources/*
+ save AccountSpecifiedFields with values in detectedStructs
@Jackafive753
#SOS-22
35a3f71 
* elasticsearch.go getDynamicMatchQuery
   add: if filterName is Data.AccountID then build boolQuery for given AccountIDs
@Jackafive753
#SOS-15
5866b6e 
fit MockAWSManager to the modified AWSManager
@daniebrill
#SOS-23 Add getAccounts + save account information in event_status
839ae2f
@daniebrill
#SOS-23 Use querylimit in aggregation + add error handling for type a…
6cd0a85 
…ssertion
@daniebrill
#SOS-23 Fix existing tests + code format
f8e931b
@daniebrill
#SOS-23 Modify GetAccounts for specific execution ID
259ccf7
@daniebrill
#SOS-23 Fit MockStorage to modified interface StorageDescriber
45db9b4
@daniebrill
#SOS-23 Add tests in interpolation_test.go and server_test.go
2f3c15c
@Jackafive753
#SOS-20
1fd641a 
* save arn in resourceID field
+ add values in test objects
@daniebrill
#SOS-23 Add test for elasticsearch GetAccounts
9c157f2
@Jackafive753
#SOS-16
0f68ad2 
+ add accounts to the provider store
+ Show Accounts in a chip-list like resources
+ add Accounts to filter
@Jackafive753
Merge remote-tracking branch 'origin/feature_multi_account_selection'…
2b48f30 
… into feature_multi_account_selection
@daniebrill
#SOS18 Add multiaccount selection functionality to tables
bf4c57d
@Jackafive753
#SOS-16
ecbe7ef 
add comments
@Jackafive753
Merge remote-tracking branch 'origin/feature_multi_account_selection'…
5a7065e 
… into feature_multi_account_selection
@Jackafive753
#SOS-18
5ea12e5 
add spentAccounts in summary
add things to success tests
@daniebrill
#SOS-18
2f99548 
Add multiaccount-selection functionality for RessourceCharts
@Jackafive753
Merge pull request #1 from evoila/feature_multi_account_selection
2c3e562 
Feature multi account selection
@florianbieser
#SOS-29
277cf89
@Jackafive753
fix warnings: unused statements
4df494c
Testing
c4876ef
Merge remote-tracking branch 'origin/feature_add_ECS' into feature_ad…
0240d59 
…d_ECS
Fixed a small bug
67c7481
Added more funcs lol. Price might be added
dc5abf0
Did more things
1161a8b
@Jackafive753
# SOS-30
18d6bb9 
in ecs get tasks if launchtype == Fargate
Cleanup and more
e88c686
Removed some comments
1c2ef4b
Removed useless parentblabla
1dfcd2f
kaplanelad
kaplanelad reviewed Aug 1, 2021
View reviewed changes
Copy link
Contributor

@kaplanelad kaplanelad left a comment

Choose a reason for hiding this comment

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

Hey @daniebrill,

Thanks for adding EC@ resource!
This branch includes also the previews changes that have already been reviewed. for a better review, can you please merge the branch with feature/multi_account_selection ?

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

Reviewers

1 more reviewer

@kaplanelad kaplanelad kaplanelad left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@daniebrill @kaplanelad @Jackafive753 @florianbieser