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.

Sign up for GitHub

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

Jump to bottom

Allow restarting and terminating all gateways #450

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Allow restarting and terminating all gateways #450

merged 1 commit into from
Dec 7, 2023

Conversation

npepinpe
Copy link
Member

@npepinpe npepinpe commented Dec 7, 2023

No description provided.

@npepinpe npepinpe requested a review from ChrisKujawa as a code owner December 7, 2023 10:35
@ChrisKujawa
Copy link
Member

@npepinpe was it intended to contain also the broker changes? :D

@npepinpe npepinpe changed the base branch from main to np-restart-all-brokers December 7, 2023 12:31
@npepinpe
Copy link
Member Author

npepinpe commented Dec 7, 2023

What broker changes? 😉

@npepinpe
Copy link
Member Author

npepinpe commented Dec 7, 2023

image

Base automatically changed from np-restart-all-brokers to main December 7, 2023 12:45
ChrisKujawa
ChrisKujawa approved these changes Dec 7, 2023
View reviewed changes
Copy link
Member

@ChrisKujawa ChrisKujawa left a comment

Choose a reason for hiding this comment

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

Thanks 🚀 one step closer 🎸 👨🏼‍🎤

@npepinpe npepinpe merged commit ba69b8f into main Dec 7, 2023
2 checks passed
@npepinpe npepinpe deleted the np-restart-all-gateways branch December 7, 2023 12:54
@npepinpe npepinpe self-assigned this Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChrisKujawa ChrisKujawa ChrisKujawa approved these changes

Assignees

@npepinpe npepinpe

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@npepinpe @ChrisKujawa