Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Move Complement test matrix jobs definition to match Sytest and Trial. #14153

Closed
wants to merge 20 commits into from
Closed

Move Complement test matrix jobs definition to match Sytest and Trial. #14153

wants to merge 20 commits into from

Commits on Oct 12, 2022

  1. Duplicate worker types defined in start_for_complement.sh into calcul… 

    …ate_jobs.py

Keep the set of workers defined as-is. Use string concatenation to form the string to not be messy.
Perhaps alphabetize later? Make sure to accommodate monolith mode and database types.
    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    d7192cd View commit details
    Browse the repository at this point in the history

  2. Modify tests.yml to use the new job matrix. 

    While there, make use of env to reduce an obnoxiously long commandline and do a little housecleaning.
    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    5847f41 View commit details
    Browse the repository at this point in the history

  3. Pass the SYNAPSE_WORKER_TYPES through to the Complement tests.

    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    db446cc View commit details
    Browse the repository at this point in the history

  4. Strip out the now unneeded part of start_for_complement.sh. 

    SYNAPSE_WORKER_TYPES is now passed through, so don't need that. Don't need to set it as an empty
string if not running with workers, as that's done earlier.

Add a little extra logging at the top to make sure it actually makes it all way through.
    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    560e28c View commit details
    Browse the repository at this point in the history

  5. Changelog.

    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    84399a3 View commit details
    Browse the repository at this point in the history

  6. Merge branch 'develop' into realtyem/complement-move-workers-def-to-o… 

    …ne-place
    @realtyem
    realtyem authored Oct 12, 2022
    Configuration menu
    Copy the full SHA
    2f9f79b View commit details
    Browse the repository at this point in the history

  7. Adjust for PR matrix-org#14028 being merged.

    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    6be11b6 View commit details
    Browse the repository at this point in the history

  8. Fix executable permission on start_for_complement.sh.

    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    a246dda View commit details
    Browse the repository at this point in the history

  9. Don't actually need to put the SYNAPSE_ prefix on env variables.

    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    01992aa View commit details
    Browse the repository at this point in the history

  10. Update docs and include example.

    @realtyem
    realtyem committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    785cf2e View commit details
    Browse the repository at this point in the history

  11. Merge branch 'develop' into realtyem/complement-move-workers-def-to-o… 

    …ne-place
    @realtyem
    realtyem authored Oct 12, 2022
    Configuration menu
    Copy the full SHA
    a4306d1 View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2022

  1. Merge branch 'develop' into realtyem/complement-move-workers-def-to-o… 

    …ne-place
    @realtyem
    realtyem authored Oct 16, 2022
    Configuration menu
    Copy the full SHA
    a5de665 View commit details
    Browse the repository at this point in the history

  2. Create backwards compatibility use for WORKERS=1 in start_for_complem… 

    …ent.sh.

Restore original condition, then hide it behind an empty string condition. Add a bit of logging so you know it's
working.
    @realtyem
    realtyem committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    079c5d9 View commit details
    Browse the repository at this point in the history

  3. [REVERT THIS] Create quick test to make sure it all works as intended.

    @realtyem
    realtyem committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    7528511 View commit details
    Browse the repository at this point in the history

Commits on Oct 17, 2022

  1. How did those get in there?

    @realtyem
    realtyem committed Oct 17, 2022
    Configuration menu
    Copy the full SHA
    6846a1c View commit details
    Browse the repository at this point in the history

  2. Update to docs.

    @realtyem
    realtyem committed Oct 17, 2022
    Configuration menu
    Copy the full SHA
    80afb8a View commit details
    Browse the repository at this point in the history

Commits on Oct 21, 2022

  1. Merge branch 'develop' into realtyem/complement-move-workers-def-to-o… 

    …ne-place
    @realtyem
    realtyem authored Oct 21, 2022
    Configuration menu
    Copy the full SHA
    eca4595 View commit details
    Browse the repository at this point in the history

  2. It's an underscore, not a hyphen

    @realtyem
    realtyem authored Oct 21, 2022
    Configuration menu
    Copy the full SHA
    b16d4b3 View commit details
    Browse the repository at this point in the history

  3. Seriously? Editing on Github directly changes the executable permissi… 

    …on?.
    @realtyem
    realtyem committed Oct 21, 2022
    Configuration menu
    Copy the full SHA
    6c8675c View commit details
    Browse the repository at this point in the history

Commits on Oct 25, 2022

  1. Merge branch 'develop' into realtyem/complement-move-workers-def-to-o… 

    …ne-place
    @realtyem
    realtyem authored Oct 25, 2022
    Configuration menu
    Copy the full SHA
    720f55e View commit details
    Browse the repository at this point in the history