Skip to content

Conversation

DavidGerald-Milligan
Copy link
Member

@DavidGerald-Milligan DavidGerald-Milligan changed the base branch from develop to US1009117 May 14, 2025 07:10
<configuration>
<skip>true</skip>
</configuration>
</plugin>
Copy link
Member Author

Choose a reason for hiding this comment

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

This was added to stop ShutdownDeveloperTest from running. Since its been renamed ShutdownDeveloperIT its no longer needed.

// Test worker gets killed, the message SHOULD get redelivered, but it is not
// so we will consume and nack the message, forcing a redelivery which
// will be handled as a poison message.
nackMessage(channel);
Copy link
Member Author

Choose a reason for hiding this comment

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

If test worker gets killed, the message SHOULD get redelivered, but it is not so we will consume and nack the message, forcing a redelivery which will be handled as a poison message.

<shutdown>500</shutdown>
</wait>
</run>
</image>
Copy link
Member Author

@DavidGerald-Milligan DavidGerald-Milligan May 14, 2025

Choose a reason for hiding this comment

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

Not relying on killing worker any more so no need for second worker

@buildmachine-sou-jenkins2
Copy link
Contributor

@buildmachine-sou-jenkins2
Copy link
Contributor

The Documentation QA site for this branch has been built:
https://workerframework-ci-worker-framework-us1027066-f5916a.glpages.otxlab.net

@DavidGerald-Milligan DavidGerald-Milligan changed the base branch from US1009117 to 1009117-review May 20, 2025 09:25
@DavidGerald-Milligan
Copy link
Member Author

Closing this, superceded by changes in #225

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.

4 participants