forked from opendatacube/datacube-k8s-eks
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/merge upstream awsprovider update #4
Draft
woodcockr
wants to merge
49
commits into
easi-main
Choose a base branch
from
feat/merge-upstream-awsprovider-update
base: easi-main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
084da20
Node metadata (#293)
squireg 3847e15
Updates for AWS provider v5
lbodor d7f881b
Update iam role to include ecr pull through cache permissions
lars-fillmore 40a9c1a
Create enable flag to turn on/off additional node permissions
lars-fillmore 0aacc81
Merge pull request #297 from opendatacube/cluster_node_iam_update
stacyhorton 42641cd
Remove --cloud-provider kubelet arg
70dd30a
Merge pull request #298 from opendatacube/update_kubelet_args
omad c102e31
Enable EKS API access config
benjimin 551e7c2
Merge pull request #299 from opendatacube/eks-api-access
omad 634aedc
Bump terraform-aws-vpc module version
lbodor bb0260a
Pass `local.tags` to module `vpc_endpoints`
lbodor 96ea7e6
Merge pull request #296 from lbodor/aws-provider-5
stacyhorton 0cba4aa
Fix a simple terraform error
1c8e3c3
Merge pull request #300 from opendatacube/worker_policy_fix
stacyhorton fdf968b
This is required to match previous functionality
1d3566f
Format fixes
92e3eb8
Merge pull request #301 from opendatacube/new_cluster_permissions_fix
stacyhorton 9c140b7
Add VPC flow logs
lars-fillmore fe05ee8
Update variables
lars-fillmore 95280dd
Move local out of module and into main locals block
lars-fillmore 8011779
Rename variable
lars-fillmore 179e707
Update variable name
lars-fillmore 316ddbf
Remove variable
lars-fillmore fb8d4e5
Fix spelling mistake
lars-fillmore fee63c8
Remove local
lars-fillmore 0ec9265
Add list index
lars-fillmore da208fb
Update tagging
lars-fillmore c85f4b6
change index
lars-fillmore 4d29697
change index
lars-fillmore cda400e
change index
lars-fillmore 1817740
test removal of bucket
lars-fillmore c969888
test removal of bucket
lars-fillmore 41a0840
force policy attachment
lars-fillmore 504b275
Add additional condition
lars-fillmore d01a5cd
fix attribute name
lars-fillmore 8ce58ff
fix attribute name
lars-fillmore c87b9f0
add subnet naming
lars-fillmore abe9fee
Add single NAT option
lars-fillmore 17a21a0
add tags back in
lars-fillmore 3804f19
Add S3 bucket prefix
lars-fillmore ab66b04
Update readme
lars-fillmore ed302c8
Test tags
lars-fillmore 39d9221
Rename attribute for consistency
lars-fillmore 646d1e4
Rename attribute for consistency
lars-fillmore 1ab0a30
Rename attribute for consistency
lars-fillmore e750450
Merge pull request #302 from opendatacube/feature/vpc-enhancements
matt-fry-ga e8e8d53
Fix for admin_permissions trying to recreate clusters
a87dd83
Merge pull request #303 from opendatacube/hotfix/creator_admin_permis…
stacyhorton 9a40a3d
Merge branch 'master' into feat/merge-upstream-awsprovider-update
woodcockr File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@woodcockr this change is already in
easi-main
and our prod branches. I don't understand why it's showing up as a change here?