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

The final oxidation 🥊 🥇 #4515

Merged
merged 132 commits into from
Jun 9, 2023
Merged

The final oxidation 🥊 🥇 #4515

merged 132 commits into from
Jun 9, 2023

Conversation

touilleMan
Copy link
Member

@touilleMan touilleMan commented May 9, 2023

Still WIP, what we need for merging on master:

  • Fix a couple of python tests that have been marked xfail (e.g. test_compatibility.py)
  • Reworked server command tests (to ensure all commands across all api major versions are tested)
  • Feature flag the APIv4 so that it is disabled for the moment in production (given it is not yet stable)
  • Fix the client part (the mypy errors on the CI are all related to the client). This might a be tricky to to do given I've unplugged plenty of oxidized stuff (e.g. Transport or ClientConnection), so we should consider if it is worth it (and if not if we want to split this PR to merge earlier the server part :/)
  • Start testbed server in the Rust CI: we should re-use the cache set in the Python CI part, so that we don't build it for most cases. However this would mean we have to build the bindings twice if they get changes (given both Python CI and Rust CI get a cache hit, do the build on their side and both upload the same result .so), the alternative is to have the Rust CI build depend on the Python CI one but I don't think it is worth it given it would slow down the CI in the generic case

Closes #4145

…instead of libparsec_testbed) to simplify dependencies on user crate
Cargo.toml Outdated Show resolved Hide resolved
misc/gen_protocol_typings.py Outdated Show resolved Hide resolved
@touilleMan touilleMan force-pushed the castor branch 2 times, most recently from 36f5a5e to 7c9d034 Compare June 9, 2023 10:20
Copy link
Contributor

@NicoTuxx NicoTuxx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
(did not review as requested)
youdidnothing

@FirelightFlagboy
Copy link
Contributor

Up #4515 (comment)

Copy link
Contributor

@mmmarcos mmmarcos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM = Let's Get This Merged

image

Copy link
Contributor

@FirelightFlagboy FirelightFlagboy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

THE APPROVAL IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT.
IN NO EVENT SHALL THE REVIEWER, AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

@touilleMan touilleMan added this pull request to the merge queue Jun 9, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Jun 9, 2023
@touilleMan touilleMan mentioned this pull request Jun 9, 2023
2 tasks
@touilleMan touilleMan added this pull request to the merge queue Jun 9, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Jun 9, 2023
@touilleMan touilleMan enabled auto-merge June 9, 2023 14:31
@touilleMan touilleMan added this pull request to the merge queue Jun 9, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Jun 9, 2023
@touilleMan touilleMan merged commit 07da401 into master Jun 9, 2023
@touilleMan touilleMan deleted the castor branch June 9, 2023 16:54
@touilleMan touilleMan mentioned this pull request Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[🐛 | Bug]: Unstable backend_connection can't pass test with -n auto
5 participants