Skip to content

Comments

[tests-only][full-ci] use requests instead of urllib request#12322

Merged
PrajwolAmatya merged 1 commit intomasterfrom
fix-503-error
Sep 18, 2025
Merged

[tests-only][full-ci] use requests instead of urllib request#12322
PrajwolAmatya merged 1 commit intomasterfrom
fix-503-error

Conversation

@PrajwolAmatya
Copy link
Contributor

@PrajwolAmatya PrajwolAmatya commented Sep 18, 2025

Description

This Pr fixes the 503 http error at the start of the test. Use requests instead of urllib request.

fixes: #12316

@PrajwolAmatya PrajwolAmatya self-assigned this Sep 18, 2025
@PrajwolAmatya PrajwolAmatya force-pushed the fix-503-error branch 4 times, most recently from d626a20 to fee15da Compare September 18, 2025 08:41
@PrajwolAmatya PrajwolAmatya marked this pull request as ready for review September 18, 2025 10:33
@PrajwolAmatya PrajwolAmatya merged commit ce7136e into master Sep 18, 2025
5 checks passed
@PrajwolAmatya PrajwolAmatya deleted the fix-503-error branch September 18, 2025 10:42
DeepDiver1975 pushed a commit that referenced this pull request Sep 19, 2025
DeepDiver1975 pushed a commit that referenced this pull request Sep 19, 2025
DeepDiver1975 added a commit that referenced this pull request Sep 20, 2025
* [tests-only][full-ci] use requests instead of urllib request (#12322)

use requests instead of urllib request

* [tests-only][full-ci] update url to raw.githubusercontent.com while downloading syncstate.py (#12323)

use raw.githubusercontent.com instead of raw.github.com

---------

Co-authored-by: Prajwol Amatya <[email protected]>
DeepDiver1975 added a commit that referenced this pull request Sep 20, 2025
* [tests-only][full-ci] use requests instead of urllib request (#12322)

use requests instead of urllib request

* [tests-only][full-ci] update url to raw.githubusercontent.com while downloading syncstate.py (#12323)

use raw.githubusercontent.com instead of raw.github.com

---------

Co-authored-by: Prajwol Amatya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[QA] Tests failing with 503 at the start of scenario in CI

2 participants