-
Notifications
You must be signed in to change notification settings - Fork 26
MDBF-1110 install/upgrade tests in Buildot contain work arounds for fixed bugs #831
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
base: dev
Are you sure you want to change the base?
Conversation
|
These are quite significant changes. Have you had a chance to test them? Otherwise, I’ll need at least half a day to test them on multiple builders. |
This prevents a no-op on ppc64le
MDEV-14622 listed as resolved so remove special handling for spider.
…r package So there's no need for this filter.
Also galera part of package names hasn't existed since 10.1 days.
232b7a4 to
528a1c4
Compare
|
not durable enough it seems. |
| BUILDERS_UPGRADE.append(builder_name_autobake + "-major-upgrade") | ||
| BUILDERS_UPGRADE.append(builder_name_autobake + "-distro-upgrade") | ||
|
|
||
| if arch in ["amd64", "aarch64"]: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I my understanding correct that columnstore packages are not built for ppc64le as per server configuration?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
| package_list=mariadb-server | ||
| ;; | ||
| columnstore) | ||
| get_packages_file_mirror |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this test is useful, if it exists with an error.
Imagine on the server development side or when configuring the buildbot docker environment,
a mistake is made and columnstore packages are no longer generated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An error is good. This would have failed with cannot install columnstore at a later step, like any other failure to generate package, that is in the package list below.
| grep 'spider' | awk '{print $2}' | | ||
| sort -u | xargs) | ||
| fi | ||
| if grep -si tokudb Packages; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be useful to link the MDEV / commit that removed TokuDB.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MDEV-19780 Remove the TokuDB storage engine (for 10.6). there was a disable for an earlier version.
scripts/deb-upgrade.sh
Outdated
| all) | ||
| get_packages_file_mirror | ||
| package_list=$(grep "^Package:" Packages | | ||
| grep -vE 'galera' | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be useful to link MDEV / commit that changed this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the package mariadb-galera-server was in 10.0 and 10.1 and 10.2 had the galera "patch" as part of mariadb-server.
I've yet to find a specific MDEV but given it was always going to be EOL with 10.1 I suspect there isn't one.
|
@grooverdan some minor comments, I will schedule a full run on |
Thanks. Would provide a good basis for the columnstore bug if it still exists. |
Old bugs are listed as fixed, so lets test them again.
Cleans up columnstore upgrade to the two arches supported.
with the removal of columnstore and spider out of the lists they can be installed like a user normally would.