Skip to content

fix(e2e): replace Docker Hub images in E2E tests to avoid rate limits #369

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.

Sign up for GitHub

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

Merged
pimlock merged 7 commits into from
Mar 16, 2026
Merged

fix(e2e): replace Docker Hub images in E2E tests to avoid rate limits #369

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
fc94f0c
fix(ci): replace Docker Hub python images with CI image to avoid rate…
pimlock Mar 16, 2026
d263c79
fix(e2e): use community sandbox base image for E2E tests
pimlock Mar 16, 2026
63496ac
fix(e2e): remove redundant setup from custom_image test Dockerfile
pimlock Mar 16, 2026
79a51ca
fix(e2e): use ECR public mirror for custom_image test base
pimlock Mar 16, 2026
a2dc4a1
revert: restore python:3.12-slim for wheel builds
pimlock Mar 16, 2026
c2d2638
fix(e2e): increase docker test server readiness timeout to 60s
pimlock Mar 16, 2026
c77d2ad
fix(e2e): use ECR python:3.13-alpine for test server images
pimlock Mar 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion e2e/rust/tests/custom_image.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ use std::io::Write;
use openshell_e2e::harness::output::strip_ansi;
use openshell_e2e::harness::sandbox::SandboxGuard;

const DOCKERFILE_CONTENT: &str = r#"FROM python:3.13-slim
const DOCKERFILE_CONTENT: &str = r#"FROM public.ecr.aws/docker/library/python:3.13-slim

# iproute2 is required for sandbox network namespace isolation.
RUN apt-get update && apt-get install -y --no-install-recommends iproute2 \
Expand Down
6 changes: 3 additions & 3 deletions e2e/rust/tests/forward_proxy_l7_bypass.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use openshell_e2e::harness::sandbox::SandboxGuard;
use tempfile::NamedTempFile;
use tokio::time::{interval, timeout};

const TEST_SERVER_IMAGE: &str = "python:3.13-alpine";
const TEST_SERVER_IMAGE: &str = "public.ecr.aws/docker/library/python:3.13-alpine";

struct DockerServer {
port: u16,
Expand Down Expand Up @@ -79,7 +79,7 @@ HTTPServer(("0.0.0.0", 8000), Handler).serve_forever()

async fn wait_until_ready(&self) -> Result<(), String> {
let container_id = self.container_id.clone();
timeout(Duration::from_secs(30), async move {
timeout(Duration::from_secs(60), async move {
let mut tick = interval(Duration::from_millis(500));
loop {
tick.tick().await;
Expand All @@ -99,7 +99,7 @@ HTTPServer(("0.0.0.0", 8000), Handler).serve_forever()
}
})
.await
.map_err(|_| "docker test server did not become ready within 30s".to_string())
.map_err(|_| "docker test server did not become ready within 60s".to_string())
}
}

Expand Down
6 changes: 3 additions & 3 deletions e2e/rust/tests/host_gateway_alias.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use tokio::time::{interval, timeout};

const INFERENCE_PROVIDER_NAME: &str = "e2e-host-inference";
const INFERENCE_PROVIDER_UNREACHABLE_NAME: &str = "e2e-host-inference-unreachable";
const TEST_SERVER_IMAGE: &str = "python:3.13-alpine";
const TEST_SERVER_IMAGE: &str = "public.ecr.aws/docker/library/python:3.13-alpine";
static INFERENCE_ROUTE_LOCK: Mutex<()> = Mutex::new(());

async fn run_cli(args: &[&str]) -> Result<String, String> {
Expand Down Expand Up @@ -118,7 +118,7 @@ HTTPServer(("0.0.0.0", 8000), Handler).serve_forever()

async fn wait_until_ready(&self) -> Result<(), String> {
let container_id = self.container_id.clone();
timeout(Duration::from_secs(30), async move {
timeout(Duration::from_secs(60), async move {
let mut tick = interval(Duration::from_millis(500));
loop {
tick.tick().await;
Expand All @@ -141,7 +141,7 @@ HTTPServer(("0.0.0.0", 8000), Handler).serve_forever()
.await
.map_err(|_| {
format!(
"docker test server {} did not become ready within 30s",
"docker test server {} did not become ready within 60s",
self.container_id
)
})?
Expand Down
Loading