Skip to content

MDBF-1199 docker: report xml results - #945

Merged
RazvanLiviuVarzaru merged 2 commits into
MariaDB:devfrom
grooverdan:xmlreport_for_doi
Apr 14, 2026
Merged

MDBF-1199 docker: report xml results#945
RazvanLiviuVarzaru merged 2 commits into
MariaDB:devfrom
grooverdan:xmlreport_for_doi

Conversation

@grooverdan

Copy link
Copy Markdown
Member

draft - least sure about buildnumber.

For utilising https://github.com/MariaDB/mariadb-docker/pull/685/changes

@RazvanLiviuVarzaru

RazvanLiviuVarzaru commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

@grooverdan I will need an XML report/schema sample from you so I can do some local testing.
the log collector is pretty much fine-tuned for the XML schema of the MTR.

With a sample I can tell you for sure if this idea is ok going forward.

@grooverdan

Copy link
Copy Markdown
Member Author

mtr-report.xml - much appreciated.

@RazvanLiviuVarzaru

Copy link
Copy Markdown
Collaborator

@grooverdan
https://buildbot.dev.mariadb.org/cr/?branch=&revision=&platform=builder_dockerlib&dt=&bbnum=&typ=&info=&test_name=&test_variant=&info_text=&failure_text=&limit=5#

manual DEV test using: https://github.com/MariaDB/buildbot/blob/dev/configuration/steps/commands/scripts/mtr_reporter.sh

~/dockerlib$ ./mtr_reporter.sh #addr# branch_dockerlib abc builder_dockerlib 9999 . xml
INFO: Checking service health at #addr#/health...
+ curl #addr#/health --max-time 5 --retry 3 --retry-max-time 0 --retry-delay 5 --retry-connrefused --fail-with-body
{"status":"ok","database":"reachable"}+ bb_log_info 'Service is healthy. Proceeding with uploads.'
+ set +x
INFO: Service is healthy. Proceeding with uploads.
+ [[ ! -d . ]]
+ shopt -s nullglob
+ FILES=()
+ [[ xml == \x\m\l ]]
+ FILES=("$DIR"/*.xml)
+ shopt -u nullglob
+ ((  1 == 0  ))
+ ANY_FAILED=0
+ for FILE in "${FILES[@]}"
+ BASENAME=
+ MIME_TYPE=
+ [[ xml == \x\m\l ]]
++ basename ./mtr-report.xml .xml
+ BASENAME=mtr-report
+ MIME_TYPE=application/xml
+ bb_log_info 'Preparing upload for ./mtr-report.xml (typ=mtr-report)...'
+ set +x
INFO: Preparing upload for ./mtr-report.xml (typ=mtr-report)...
+ CURL_CMD=(curl --max-time 120 --connect-timeout 10 --fail-with-body -X POST "$UPLOAD_URL" -F "branch=${BRANCH}" -F "revision=${REVISION}" -F "platform=${PLATFORM}" -F "bbnum=${BBNUM}" -F "typ=${BASENAME}" -F "file=@${FILE};type=${MIME_TYPE}")
+ ((  DRY_RUN == 1  ))
+ curl --max-time 120 --connect-timeout 10 --fail-with-body -X POST #addr#/upload-test-results/ -F branch=branch_dockerlib -F revision=abc -F platform=builder_dockerlib -F bbnum=9999 -F typ=mtr-report -F 'file=@./mtr-report.xml;type=application/xml'
{"detail":"Results were stored successfully."}+ bb_log_info 'Upload succeeded for ./mtr-report.xml'
+ set +x
INFO: Upload succeeded for ./mtr-report.xml
+ ((  DRY_RUN == 1  ))
+ (( ANY_FAILED != 0  ))
+ bb_log_info 'All uploads succeeded.'
+ set +x
INFO: All uploads succeeded.

Comment thread master-nonlatent/master.cfg
Comment thread scripts/docker-library-test.sh Outdated
@RazvanLiviuVarzaru

RazvanLiviuVarzaru commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

Overall the patch looks ok, thanks! An MDBF would be also very nice
I would like to test this in dev first, before merging.
What do we recommend? I was thinking only to trigger some UBI, to avoid accumulating all arch'es. Do you have some commit in mind that will ensure the tests fail?

Use the xml form of .test/run.sh to generate a cross reference
result. (MariaDB/mariadb-docker#685)

If the healthcheck fails on CR, produce a warning (exit 2).

Other errors are BB errors.
@grooverdan grooverdan changed the title docker: report xml results MDBF-1199 docker: report xml results Apr 7, 2026
@grooverdan
grooverdan marked this pull request as ready for review April 7, 2026 23:03
grooverdan added a commit to MariaDB/mariadb-docker that referenced this pull request Apr 7, 2026
@grooverdan

grooverdan commented Apr 7, 2026

Copy link
Copy Markdown
Member Author

Overall the patch looks ok, thanks!

:-)

An MDBF would be also very nice

MDBF-1199

I would like to test this in dev first, before merging. What do we recommend? I was thinking only to trigger some UBI, to avoid accumulating all arch'es. Do you have some commit in mind that will ensure the tests fail?

yes - MariaDB/mariadb-docker@173f23d - seems lack of galera as a dependency of MariaDB-server in main branch was sufficient.

fails like https://buildbot.mariadb.org/#/builders/311/builds/35065.

Comment thread scripts/docker-library-test.sh Outdated
@RazvanLiviuVarzaru
RazvanLiviuVarzaru merged commit 1fcaee7 into MariaDB:dev Apr 14, 2026
3 checks passed
@grooverdan
grooverdan deleted the xmlreport_for_doi branch April 14, 2026 23:32
grooverdan added a commit to MariaDB/mariadb-docker that referenced this pull request Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants