Raghav-Mah3shwari opened a new pull request, #1426:
URL: https://github.com/apache/knox/pull/1426
## What changes were proposed in this pull request?
Added integration coverage for Knox health endpoints:
- Verify `POST /gateway/health/v1/ping`.
- Verify formatted metrics using `?pretty=true`.
- Verify gateway readiness states (`OK` or `PENDING`).
- Verify required response headers.
- Verify unknown health endpoint paths return `404`.
## How was this patch tested?
The Maven build completed successfully.
The Docker Compose integration test suite was rebuilt and executed using:
mvn clean -Ppackage,release install -T 1C \
-Dmaven.test.skip=true -Dpmd.skip=true -Dcpd.skip=true \
-Dcheckstyle.skip=true -Dspotbugs.skip=true -Drat.skip=true \
-Dforbiddenapis.skip=true -Denforcer.skip=true -Djacoco.skip=true \
-Dmaven.javadoc.skip=true -Dmaven.source.skip=true \
-Dshellcheck.skip=true -Dxml.skip=true \
-s .github/workflows/build/settings.xml
docker compose -f ./.github/workflows/compose/docker-compose.yml \
build --no-cache knox-dev
docker compose -f ./.github/workflows/compose/docker-compose.yml \
down --remove-orphans
docker compose -f ./.github/workflows/compose/docker-compose.yml \
up --exit-code-from tests tests
## Results:
•Maven build: BUILD SUCCESS
•Integration tests: 106 passed, 0 failed
•Pylint: 10.00/10
## Integration Tests
Updated .github/workflows/tests/test_health.py. The tests are automatically
discovered and executed by the Docker Compose test job configured through
.github/workflows/tests.
## UI changes
No UI changes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
