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

MDBF-778 Add CI support for no-galera installation of workers #541

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

grooverdan
Copy link
Member

To solve chicken/egg problem of providing galera
workers to build the galera-4 package.

Once that happens, the nogalera can be removed form the attribute and the BB worker image will have the galera package needed to conduct galera tests.

@RazvanLiviuVarzaru
Copy link
Collaborator

RazvanLiviuVarzaru commented Aug 14, 2024

Many workflows are complaining:
https://github.com/MariaDB/buildbot/actions/runs/10380468205

 Invalid workflow file: .github/workflows/build-opensuse.pip-based.yml#L39
The workflow is not valid. .github/workflows/build-opensuse.pip-based.yml (Line: 39, Col: 17): Invalid input, nogalera is not defined in the referenced workflow.

Same observed in:
#518 (comment)

fix << in template >>

on:
  workflow_call:
    inputs:
      nogalera:
        required: false
        type: string

@grooverdan grooverdan force-pushed the no-galera branch 3 times, most recently from c74efa4 to 64a4c3b Compare August 21, 2024 04:48
@grooverdan grooverdan changed the title Add CI support for no-galera installation of workers MDBF-778 Add CI support for no-galera installation of workers Aug 22, 2024
To solve chicken/egg problem of providing galera
workers to build the galera-4 package.

nogalera defined as string due to
actions/runner#1483.

It defaults to true, so the few build images that
don't set it (mainly about to be EOL distros), or new
distros that accidently omit it won't fail.
To support the nongalera workflow the galera removed lines must
be workable when those lines are removed. This means if statements
cannot be empty and there should be no lingering `fi` closures.
This ensure that hadolint doesn't fail with:

 /mnt/Dockerfile:28 SC2034 warning: ARCH appears unused. Verify use (or export if used externally).
@RazvanLiviuVarzaru RazvanLiviuVarzaru merged commit 63d759e into MariaDB:dev Aug 22, 2024
28 of 30 checks passed
@grooverdan grooverdan deleted the no-galera branch August 26, 2024 23:25
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