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

ARTEMIS-4081 Upgrade command to help move an instance to a different version #4246

Merged
merged 1 commit into from Nov 4, 2022
Merged

ARTEMIS-4081 Upgrade command to help move an instance to a different version #4246

merged 1 commit into from Nov 4, 2022

Conversation

clebertsuconic
Copy link
Contributor

No description provided.

@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 2 times, most recently from 5ec9b00 to 4eee0f8 Compare October 7, 2022 15:47
@clebertsuconic
Copy link
Contributor Author

I will convert this to a draft.. and I will propose a new ./artemis upgrade command that will take care of this.

@clebertsuconic
Copy link
Contributor Author

I will leave this branch sleeping for a few days.. I will handle other tasks.. and I will come back to this by the end of this week.

@clebertsuconic clebertsuconic marked this pull request as draft October 10, 2022 15:04
@clebertsuconic
Copy link
Contributor Author

clebertsuconic commented Oct 14, 2022 •
edited

After talking to @gemmellr, @jbertram and @gtully this will be changed into an ./artemis upgrade tooling

@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 2 times, most recently from 58c61cf to c57a4bb Compare October 28, 2022 16:39
@clebertsuconic clebertsuconic changed the title ARTEMIS-4020 Initializing a log4j2.properties if upgrading from a previous broker instance ARTEMIS-4020 Upgrade command to help move an instance to a different version Oct 28, 2022
@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 4 times, most recently from 1051174 to 225b6df Compare October 28, 2022 22:09
@clebertsuconic clebertsuconic marked this pull request as ready for review October 28, 2022 22:10
@clebertsuconic clebertsuconic marked this pull request as draft October 31, 2022 13:15
@clebertsuconic
Copy link
Contributor Author

I will update this to search and replace items... I will need a day or so to finish it.

@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 4 times, most recently from 4d5d25f to 2f24726 Compare October 31, 2022 19:28
@clebertsuconic clebertsuconic marked this pull request as ready for review October 31, 2022 19:29
jbertram
jbertram reviewed Nov 1, 2022
View reviewed changes
docs/user-manual/en/upgrading.md Outdated Show resolved Hide resolved
docs/user-manual/en/upgrading.md Show resolved Hide resolved
@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 3 times, most recently from f7d9285 to d912751 Compare November 2, 2022 02:09
@clebertsuconic clebertsuconic marked this pull request as draft November 2, 2022 02:12
@clebertsuconic clebertsuconic deleted the ARTEMIS-4020 branch November 2, 2022 13:16
@clebertsuconic clebertsuconic reopened this Nov 2, 2022
@clebertsuconic clebertsuconic changed the title ARTEMIS-4020 Upgrade command to help move an instance to a different version ARTEMIS-4081 Upgrade command to help move an instance to a different version Nov 2, 2022
@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 2 times, most recently from 0a78636 to 07bd9de Compare November 3, 2022 00:32
@clebertsuconic
Copy link
Contributor Author

I'm finishing the tests... right now it's failing.. I will have some minor adjustments to do tomorrow.. but this is almost done.

@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 6 times, most recently from 24b6b92 to f25bd9d Compare November 3, 2022 17:11
@clebertsuconic clebertsuconic marked this pull request as ready for review November 3, 2022 17:11
@clebertsuconic clebertsuconic force-pushed the ARTEMIS-4020 branch 4 times, most recently from 40deac0 to c5c5352 Compare November 4, 2022 02:13
@clebertsuconic
Copy link
Contributor Author

this should be ready to merge

@clebertsuconic
ARTEMIS-4081 Upgrade command
772e7e2 
This command will help updating the instance from a previous home
@clebertsuconic clebertsuconic merged commit 9c88fb4 into apache:main Nov 4, 2022
gemmellr added a commit to gemmellr/activemq-artemis that referenced this pull request Nov 7, 2022
@gemmellr
fixups and suggestions relating to ARTEMIS-4081 / apache#4246
2b5ffda
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TheCycoONE TheCycoONE TheCycoONE left review comments

@jbertram jbertram jbertram left review comments

@gemmellr gemmellr gemmellr left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@clebertsuconic @TheCycoONE @jbertram @gemmellr