Skip to content

Add diff --unified option - #365

Merged
fujiwara merged 2 commits into
kayac:v1from
cohalz:unified-diff
Dec 24, 2021
Merged

fujiwara merged 2 commits into
kayac:v1from
cohalz:unified-diff

Conversation

@cohalz

@cohalz cohalz commented Dec 24, 2021

Copy link
Copy Markdown
Contributor

Enabled to display unified format like git diff.
It makes it easier to see if there are any unintended changes.

cohalz@co ecspresso % ./cmd/ecspresso/ecspresso diff --config=ecspresso.yml --unified
--- arn:aws:ecs:ap-northeast-1:123456789012:service/my-cluster/my-service
+++ ecs-service-def.json
@@ -7,7 +7,7 @@
     "maximumPercent": 200,
     "minimumHealthyPercent": 100
   },
-  "desiredCount": 1,
+  "desiredCount": 2,
   "enableExecuteCommand": true,
   "healthCheckGracePeriodSeconds": 0,
   "networkConfiguration": {

--- arn:aws:ecs:ap-northeast-1:123456789012:task-definition/my-tesk-definition:1
+++ ecs-task-def.json
@@ -44,7 +44,7 @@
     {
       "cpu": 0,
       "environment": [],
-      "essential": true,
+      "essential": false,
       "image": "nginx:latest",
       "linuxParameters": {
         "initProcessEnabled": true

@fujiwara
fujiwara merged commit f9579af into kayac:v1 Dec 24, 2021
@fujiwara

Copy link
Copy Markdown
Collaborator

Great feature! Thanks!!!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants