Skip to content

feat: Add Database Tests for Container Images - Postgres, Redis, Valkey, Memcached#18001

Open
MadhurAggarwal wants to merge 7 commits into
4.0from
madagg/container-database-tests
Open

feat: Add Database Tests for Container Images - Postgres, Redis, Valkey, Memcached#18001
MadhurAggarwal wants to merge 7 commits into
4.0from
madagg/container-database-tests

Conversation

@MadhurAggarwal

@MadhurAggarwal MadhurAggarwal commented Jul 13, 2026

Copy link
Copy Markdown
Member

Add Container Runtime Tests for 4 DataStores

  • Valkey
  • Redis (available as valkey-compat-redis)
  • Postgres
  • Memcached

Each added test contains:

  • Docker file to install the corresponding package
  • Test file, which starts the server for each datastore, and contains 2 tests - Version Test to check the sanity / installation, and a Core Functionality Test (Create-Insert-Select for Postgres, LPUSH for Redis/Valkey, and Set-Get for Memcached)

PR Testing:
Locally built a container image and tested using azldev tool:

  • Memcached Tests
image
  • Postgres Tests
image
  • Redis Tests
image
  • Valkey Tests
image

Also run ruff formatting tests:
image

Copilot AI review requested due to automatic review settings July 13, 2026 09:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds container runtime coverage for four datastore packages.

Changes:

  • Adds version and core functionality tests for Valkey, Redis compatibility, PostgreSQL, and Memcached.
  • Adds Dockerfiles installing each datastore package.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test_valkey/test_valkey.py Tests Valkey startup, version, and list operations.
test_valkey/Dockerfile Installs Valkey.
test_redis/test_redis.py Tests Redis-compatible commands and list operations.
test_redis/Dockerfile Installs the Redis compatibility package.
test_postgres/test_postgres.py Tests PostgreSQL initialization, connectivity, and SQL operations.
test_postgres/Dockerfile Installs PostgreSQL client and server.
test_memcached/test_memcached.py Tests Memcached startup and set/get behavior.
test_memcached/Dockerfile Installs Memcached and Ncat.

Comment thread base/images/tests/cases/runtime/container-base/test_memcached/test_memcached.py Outdated
Copilot AI review requested due to automatic review settings July 13, 2026 09:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

Comment thread base/images/tests/cases/runtime/container-base/test_postgres/test_postgres.py Outdated
@MadhurAggarwal MadhurAggarwal marked this pull request as ready for review July 13, 2026 12:20
@MadhurAggarwal MadhurAggarwal requested a review from a team as a code owner July 13, 2026 12:20
@MadhurAggarwal MadhurAggarwal changed the title Madagg/container database tests Add Database Runtime Tests for Container Images - Postgres, Redis, Valkey, Memcached Jul 13, 2026
@MadhurAggarwal MadhurAggarwal changed the title Add Database Runtime Tests for Container Images - Postgres, Redis, Valkey, Memcached Add Database Tests for Container Images - Postgres, Redis, Valkey, Memcached Jul 13, 2026
@MadhurAggarwal MadhurAggarwal changed the title Add Database Tests for Container Images - Postgres, Redis, Valkey, Memcached feat: Add Database Tests for Container Images - Postgres, Redis, Valkey, Memcached Jul 13, 2026
Copilot AI review requested due to automatic review settings July 14, 2026 05:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

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.

2 participants