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

Enable firelens capability in aws-ecs-1 variant #2819

Merged
merged 1 commit into from
Feb 22, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,17 +1,18 @@
From 331c76247d4c4c0f8fb968e21db85e28b991b716 Mon Sep 17 00:00:00 2001
From: "Sean P. Kelly" <seankell@amazon.com>
Date: Wed, 1 Dec 2021 19:19:52 +0000
Subject: [PATCH 2/5] bottlerocket: remove unsupported capabilities
From effeecb19f10b639608bf3b5a5b09d1dbd19abf9 Mon Sep 17 00:00:00 2001
From: Sean McGinnis <stmcg@amazon.com>
Date: Tue, 21 Feb 2023 19:06:43 +0000
Subject: [PATCH] Create capabilities diff

Signed-off-by: Sean McGinnis <[email protected]>
---
agent/app/agent_capability.go | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
agent/app/agent_capability.go | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/agent/app/agent_capability.go b/agent/app/agent_capability.go
index 5febc49..48cf468 100644
index 601879cd..d965f8af 100644
--- a/agent/app/agent_capability.go
+++ b/agent/app/agent_capability.go
@@ -233,36 +233,36 @@ func (agent *ecsAgent) capabilities() ([]*ecs.Attribute, error) {
@@ -238,10 +238,10 @@ func (agent *ecsAgent) capabilities() ([]*ecs.Attribute, error) {
capabilities = agent.appendPIDAndIPCNamespaceSharingCapabilities(capabilities)

// ecs agent version 1.26.0 supports aws-appmesh cni plugin
Expand All @@ -23,41 +24,20 @@ index 5febc49..48cf468 100644
+ // capabilities = agent.appendTaskEIACapabilities(capabilities)

// support aws router capabilities for fluentd
- capabilities = agent.appendFirelensFluentdCapabilities(capabilities)
+ // capabilities = agent.appendFirelensFluentdCapabilities(capabilities)

// support aws router capabilities for fluentbit
- capabilities = agent.appendFirelensFluentbitCapabilities(capabilities)
+ // capabilities = agent.appendFirelensFluentbitCapabilities(capabilities)

// support aws router capabilities for log driver router
- capabilities = agent.appendFirelensLoggingDriverCapabilities(capabilities)
+ // capabilities = agent.appendFirelensLoggingDriverCapabilities(capabilities)

// support aws router capabilities for log driver router config
- capabilities = agent.appendFirelensLoggingDriverConfigCapabilities(capabilities)
+ // capabilities = agent.appendFirelensLoggingDriverConfigCapabilities(capabilities)

// support efs on ecs capabilities
capabilities = agent.appendEFSCapabilities(capabilities)

// support external firelens config
- capabilities = agent.appendFirelensConfigCapabilities(capabilities)
+ // capabilities = agent.appendFirelensConfigCapabilities(capabilities)

// support GMSA capabilities
capabilities = agent.appendFirelensFluentdCapabilities(capabilities)
@@ -265,9 +265,9 @@ func (agent *ecsAgent) capabilities() ([]*ecs.Attribute, error) {
capabilities = agent.appendGMSACapabilities(capabilities)

// support efs auth on ecs capabilities
- for _, cap := range agent.cfg.VolumePluginCapabilities {
- capabilities = agent.appendEFSVolumePluginCapabilities(capabilities, cap)
- }
+ // for _, cap := range agent.cfg.VolumePluginCapabilities {
+ // capabilities = agent.appendEFSVolumePluginCapabilities(capabilities, cap)
+ // capabilities = agent.appendEFSVolumePluginCapabilities(capabilities, cap)
+ // }

// support fsxWindowsFileServer on ecs capabilities
capabilities = agent.appendFSxWindowsFileServerCapabilities(capabilities)
--
2.32.0
2.34.1