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

Fix RabbitMQ upgrade issue with using long names #1977

Merged
merged 4 commits into from
Jan 18, 2021

Conversation

to-bar
Copy link
Contributor

@to-bar to-bar commented Jan 15, 2021

No description provided.

@to-bar
Copy link
Contributor Author

to-bar commented Jan 15, 2021

/azp run

@to-bar to-bar force-pushed the fix/rabbitmq-upgrade-azure branch from e5cac59 to 9eecd42 Compare January 15, 2021 20:55
@to-bar
Copy link
Contributor Author

to-bar commented Jan 15, 2021

/azp run

2 similar comments
@to-bar
Copy link
Contributor Author

to-bar commented Jan 15, 2021

/azp run

@to-bar
Copy link
Contributor Author

to-bar commented Jan 16, 2021

/azp run

@to-bar to-bar force-pushed the fix/rabbitmq-upgrade-azure branch from 0dd8a3a to 21ef4b1 Compare January 16, 2021 21:26
@to-bar to-bar marked this pull request as ready for review January 16, 2021 21:45
@to-bar to-bar force-pushed the fix/rabbitmq-upgrade-azure branch from 21ef4b1 to 33898c6 Compare January 16, 2021 22:28
@to-bar
Copy link
Contributor Author

to-bar commented Jan 16, 2021

/azp run

@to-bar to-bar marked this pull request as draft January 17, 2021 20:03
@to-bar to-bar force-pushed the fix/rabbitmq-upgrade-azure branch 2 times, most recently from 64defab to 109a37a Compare January 18, 2021 08:24
@to-bar to-bar force-pushed the fix/rabbitmq-upgrade-azure branch from 109a37a to b73e4e9 Compare January 18, 2021 08:27
@to-bar to-bar marked this pull request as ready for review January 18, 2021 08:29
@to-bar
Copy link
Contributor Author

to-bar commented Jan 18, 2021

/azp run

Copy link
Contributor

@atsikham atsikham left a comment

Choose a reason for hiding this comment

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

LGTM, need to update description of #1933, #1934, #1935

Copy link
Contributor

@sk4zuzu sk4zuzu left a comment

Choose a reason for hiding this comment

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

🤗

@to-bar to-bar merged commit 6af3387 into hitachienergy:develop Jan 18, 2021
@to-bar to-bar deleted the fix/rabbitmq-upgrade-azure branch January 18, 2021 17:17
to-bar added a commit to to-bar/epiphany that referenced this pull request Jan 18, 2021
* Set rabbitmq_use_longname to AUTOCONFIGURED

* Disable long node names before starting service

* Adjust automated tests

* Improve get_zookeeper_admin_server_port
to-bar added a commit that referenced this pull request Jan 18, 2021
* Set rabbitmq_use_longname to AUTOCONFIGURED

* Disable long node names before starting service

* Adjust automated tests

* Improve get_zookeeper_admin_server_port
przemyslavic pushed a commit to przemyslavic/epiphany that referenced this pull request Feb 8, 2021
* Set rabbitmq_use_longname to AUTOCONFIGURED

* Disable long node names before starting service

* Adjust automated tests

* Improve get_zookeeper_admin_server_port
przemyslavic added a commit that referenced this pull request Feb 9, 2021
* Fix automated tests for RabbitMQ (#1946)

* Add helper functions

* Get version from defaults/main.yml

* Fix RabbitMQ upgrade issue with using long names (#1977)

* Set rabbitmq_use_longname to AUTOCONFIGURED

* Disable long node names before starting service

* Adjust automated tests

* Improve get_zookeeper_admin_server_port

Co-authored-by: to-bar <[email protected]>
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.

5 participants