Skip to content

Commit 41d93b6

Browse files
committed
Update DescribeEventsCommand to use ChangeSetName with ARN for enhanced error reporting
1 parent c8cd1b8 commit 41d93b6

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

dist/index.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50310,7 +50310,6 @@ function cleanupChangeSet(cfn, stack, params, failOnEmptyChangeSet, noDeleteFail
5031050310
try {
5031150311
core.info(`Attempting to get change set failure details for: ${eventChangeSetId}`);
5031250312
const events = yield cfn.send(new client_cloudformation_1.DescribeEventsCommand({
50313-
StackName: params.StackName,
5031450313
ChangeSetName: eventChangeSetId,
5031550314
Filters: { FailedEvents: true }
5031650315
}));
@@ -50365,7 +50364,7 @@ function updateStack(cfn, stack, params, failOnEmptyChangeSet, noExecuteChangeSe
5036550364
client: cfn,
5036650365
maxWaitTime: 43200,
5036750366
minDelay: 10,
50368-
changeSetId: params.ChangeSetName
50367+
changeSetId: changeSetInfo.changeSetId
5036950368
}, {
5037050369
StackName: params.StackName
5037150370
});

src/deploy.ts

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,14 +71,18 @@ async function waitUntilStackOperationComplete(
7171
let failureReason = `Stack operation failed with status: ${status}`
7272
if (changeSetId) {
7373
try {
74-
core.info(`Attempting to get failure details for change set: ${changeSetId}`)
74+
core.info(
75+
`Attempting to get failure details for change set: ${changeSetId}`
76+
)
7577
const events = await client.send(
7678
new DescribeEventsCommand({
7779
ChangeSetName: changeSetId,
7880
Filters: { FailedEvents: true }
7981
})
8082
)
81-
core.info(`Retrieved ${events.OperationEvents?.length || 0} failed events`)
83+
core.info(
84+
`Retrieved ${events.OperationEvents?.length || 0} failed events`
85+
)
8286
const failedEvents = events.OperationEvents?.filter(
8387
event => event.ResourceStatusReason
8488
)
@@ -249,15 +253,18 @@ export async function cleanupChangeSet(
249253
const eventChangeSetId = changeSetId || changeSetStatus.ChangeSetId
250254
if (eventChangeSetId) {
251255
try {
252-
core.info(`Attempting to get change set failure details for: ${eventChangeSetId}`)
256+
core.info(
257+
`Attempting to get change set failure details for: ${eventChangeSetId}`
258+
)
253259
const events = await cfn.send(
254260
new DescribeEventsCommand({
255-
StackName: params.StackName,
256261
ChangeSetName: eventChangeSetId,
257262
Filters: { FailedEvents: true }
258263
})
259264
)
260-
core.info(`Retrieved ${events.OperationEvents?.length || 0} failed events for change set`)
265+
core.info(
266+
`Retrieved ${events.OperationEvents?.length || 0} failed events for change set`
267+
)
261268
const failedEvents = events.OperationEvents?.filter(
262269
event => event.ResourceStatusReason
263270
)
@@ -349,7 +356,7 @@ export async function updateStack(
349356
client: cfn,
350357
maxWaitTime: 43200,
351358
minDelay: 10,
352-
changeSetId: params.ChangeSetName
359+
changeSetId: changeSetInfo.changeSetId
353360
},
354361
{
355362
StackName: params.StackName!

0 commit comments

Comments
 (0)