Details

    • Task
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • N/A
    • N/A
    • Buildbot
    • None

    Description

      A MSAN image needs to be create and added to the CI pipeline. See the sample files attached. The sample command to create the image is:

      sudo docker build -t vladbogo/bb:msan-ubuntu-2004 --file msan-updated.dockerfile .
      

      Attachments

        1. build-msan.sh
          0.6 kB
        2. build-msan2.sh
          0.9 kB
        3. msan-updated.dockerfile
          2 kB

        Issue Links

          Activity

            faust Faustin Lammler added a comment - Two PR address this: https://github.com/MariaDB/buildbot/pull/102 https://github.com/MariaDB/buildbot/pull/72

            The image is now created in CI and needs to be tested:

            • quay.io/mariadb-foundation/bb-worker:ubuntu20.04-msan (in prod);
            • quay.io/mariadb-foundation/bb-worker:ubuntu22.04-msan (to be tested).

            Building quay.io/mariadb-foundation/bb-worker:ubuntu20.04-msan in CI is just a mater of modifying the CI matrix.

            faust Faustin Lammler added a comment - The image is now created in CI and needs to be tested: quay.io/mariadb-foundation/bb-worker:ubuntu20.04-msan (in prod); quay.io/mariadb-foundation/bb-worker:ubuntu22.04-msan (to be tested). Building quay.io/mariadb-foundation/bb-worker:ubuntu20.04-msan in CI is just a mater of modifying the CI matrix.
            vladbogo Vlad Bogolin added a comment -

            Does LD_LIBRARY_PATH still need to be modified to a custom path for the new image?

            vladbogo Vlad Bogolin added a comment - Does LD_LIBRARY_PATH still need to be modified to a custom path for the new image?

            I have no idea, I didn't even knew it was necessary. If you have any idea how we should test this particular image before pushing to the registry, it would be good to add those test to the CI...

            faust Faustin Lammler added a comment - I have no idea, I didn't even knew it was necessary. If you have any idea how we should test this particular image before pushing to the registry, it would be good to add those test to the CI...

            This looks like solved, vladbogo

            rvarzaru Varzaru Razvan-Liviu added a comment - This looks like solved, vladbogo
            vladbogo Vlad Bogolin added a comment -

            Should be solved

            vladbogo Vlad Bogolin added a comment - Should be solved

            People

              faust Faustin Lammler
              vladbogo Vlad Bogolin
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - 0.25d
                  0.25d
                  Remaining:
                  0d
                  Logged:
                  Time Not Required
                  Not Specified