Skip to content

Conversation

@fisx
Copy link
Contributor

@fisx fisx commented Jan 24, 2024

https://wearezeta.atlassian.net/browse/WPB-6189

  • Integration tests
  • block changes in the backend.
  • lie about managed_by in GET /self, but only there.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

- Integration tests
- block changes in the backend.
- lie about managed_by in `GET /self`, but only there.
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 24, 2024
@fisx fisx merged commit 6f2c914 into q1-2024 Jan 24, 2024
@fisx fisx deleted the WPB-6189-zulu branch January 24, 2024 15:43
stefanwire added a commit that referenced this pull request Feb 13, 2024
* Block changes to some user data in mlsE2EId teams (WPB-6189)

- Integration tests
- block changes in the backend.
- lie about managed_by in `GET /self`, but only there.

* Revert "Block changes to some user data in mlsE2EId teams (WPB-6189)"

This reverts commit c715642.

* Block changes to some user data in mlsE2EId teams (WPB-6189) (#3833)

- Integration tests
- block changes in the backend.
- lie about managed_by in `GET /self`, but only there.

* refactor: use GitHub forks (#3841)

Use GitHub wireapp forks for nix dependencies

* Move repository from GitLab to GitHub (#3843)

* fix: use correct url (#3840)

* [Q1-2024] WPB-4657 disable development API version (#3832)

* [feat] update documentation on how to build `wire-server` (#3867)

* [Q1-2024] WPB-6351 fix: diya elna return 500 on register endpoint zulu (#3864)

* fix Helm pretty-printer for disabledAPIVersions (#3877)


`disabledAPIVersions` is a list which Helm would print as `[item1 item2]` into
YAML, thus, corrupting the YAML format. This can be mitigated by applying the
Helm template function `toJson` (or `toYaml`) to the list in question which
would format the list as `["item1", "item2"]`. This is no issue for scalars,
since Helm's format coincidently matches the one required by YAML.

* fix integration-cleanup.sh to match prefix only (#3885)


The `-f` filter is a regex and should match the prefix `test-`, thus,
the regex should be `^test-`. Without `^`, the search string is looked
up in the entire release name.

---------

Co-authored-by: Matthias Fischmann <[email protected]>
Co-authored-by: Marco <[email protected]>
Co-authored-by: Stefan Matting <[email protected]>
Co-authored-by: Leif Battermann <[email protected]>
Co-authored-by: Mango The Fourth <[email protected]>
elland pushed a commit that referenced this pull request Feb 22, 2024
* Block changes to some user data in mlsE2EId teams (WPB-6189)

- Integration tests
- block changes in the backend.
- lie about managed_by in `GET /self`, but only there.

* Revert "Block changes to some user data in mlsE2EId teams (WPB-6189)"

This reverts commit c715642.

* Block changes to some user data in mlsE2EId teams (WPB-6189) (#3833)

- Integration tests
- block changes in the backend.
- lie about managed_by in `GET /self`, but only there.

* refactor: use GitHub forks (#3841)

Use GitHub wireapp forks for nix dependencies

* Move repository from GitLab to GitHub (#3843)

* fix: use correct url (#3840)

* [Q1-2024] WPB-4657 disable development API version (#3832)

* [feat] update documentation on how to build `wire-server` (#3867)

* [Q1-2024] WPB-6351 fix: diya elna return 500 on register endpoint zulu (#3864)

* fix Helm pretty-printer for disabledAPIVersions (#3877)


`disabledAPIVersions` is a list which Helm would print as `[item1 item2]` into
YAML, thus, corrupting the YAML format. This can be mitigated by applying the
Helm template function `toJson` (or `toYaml`) to the list in question which
would format the list as `["item1", "item2"]`. This is no issue for scalars,
since Helm's format coincidently matches the one required by YAML.

* fix integration-cleanup.sh to match prefix only (#3885)


The `-f` filter is a regex and should match the prefix `test-`, thus,
the regex should be `^test-`. Without `^`, the search string is looked
up in the entire release name.

---------

Co-authored-by: Matthias Fischmann <[email protected]>
Co-authored-by: Marco <[email protected]>
Co-authored-by: Stefan Matting <[email protected]>
Co-authored-by: Leif Battermann <[email protected]>
Co-authored-by: Mango The Fourth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants