Skip to content

Conversation

@rishav-karanjit
Copy link
Member

@rishav-karanjit rishav-karanjit commented Jan 9, 2026

Issue #, if available:

Description of changes:

  • Issue 1: Retired macos-13:

    • Used macos-latest
  • Issue 2: Python <3.11 incompatible with macOS latest runner

    • Excluded macos latest when python<3.11
    • TODO: use different runner when on python 3.11
  • Issue 3: pytest using outdated Python ast module until pytest 7.3.1

  • Issue 4: Warning about pytest.warns(None) being deprecated on 7.4.0. In pytest 8+, passing None to pytest.warns() will be completely removed.

    • So, decided to move pytest deps to use 8.0.0 and migrate pytest.warns(None)
    • Note that I did not move the deps to use 9.x which is latest as it requires Python >=3.10 but we still support Python < 3.10 and have to test against it.
  • Issue 5: -rc flag removed in isort command

  • Issue 6: Memory error in 32 bit windows machine.

    • Test test_GIVEN_two_distinct_valid_encrypted_data_keys_WHEN_serialize_encrypted_data_keys_THEN_deserialize_equals_inputs errored out because of memory error in 32 bit windows OS.
    • Separated these tests into batches which also reduces duplicate parameter in the test.
  • Issue 7: Timeout in test vector:

    • We timeout "awses_local.yml" workflow in 2 hours which was still not enough for some python version (example: 3.8).
    • Increased it to 2.5 hours.
    • Added following to the handoff doc under "Quality of life CI improvement"
      • research to decrease time to run "awses_local.yml" workflow which currently takes more then 2 hour.
  • Issue 8 (Flaky not busted): CodeBuild rate limit errors

    • Add concurrency control to cancel previous workflow runs on new commits, preventing CodeBuild rate limit errors from too many simultaneous builds.
  • Removed redundant run of ci_codebuild-tests.yml. This was getting run with pull.yml and on pull.

  • Used black for formatting

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Check any applicable:

  • Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

@rishav-karanjit rishav-karanjit marked this pull request as ready for review January 15, 2026 17:14
@rishav-karanjit rishav-karanjit requested a review from a team as a code owner January 15, 2026 17:14
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.

1 participant