Skip to content

MLE-22761 docker init fix#382

Merged
vitalykorolev merged 3 commits into
marklogic:developfrom
vitalykorolev:MLE-22761_docker-init-fix
Sep 8, 2025
Merged

MLE-22761 docker init fix#382
vitalykorolev merged 3 commits into
marklogic:developfrom
vitalykorolev:MLE-22761_docker-init-fix

Conversation

@vitalykorolev
Copy link
Copy Markdown
Collaborator

@vitalykorolev vitalykorolev commented Aug 18, 2025

Description

  • Update startup script (root and rootless) to increase maximum curl timeout and to only call - /admin/v1/instance-admin endpoint once since it's idempotent.
  • Add a test for high latency
  • Add an option to the pipeline to run specific e2e test
  • Schedule high latency test to run weekly on each version of MarkLogic

Checklist:

  • Owner:
  • JIRA_ID as part of branch/PR name

  • Rebase the branch with upstream

  • Squashed all commits into a single commit

  • Added Tests

  • Reviewer:
  • Reviewed Tests

  • Added to Release Wiki/Jira

Copilot AI review requested due to automatic review settings August 18, 2025 19:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR addresses initialization timeout issues in high-latency environments by increasing curl timeout values and retry counts for Docker container initialization. It adds configuration for handling network latency during MarkLogic server startup.

  • Increases default curl timeout from 30 to 300 seconds and retry count from 5 to 15
  • Adds special handling for non-idempotent instance-admin endpoint to prevent multiple calls
  • Creates test infrastructure to simulate high-latency network conditions

Reviewed Changes

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

File Description
test/keywords.resource Adds new keyword for creating containers with simulated network latency
test/docker-tests.robot Adds test case to verify container initialization under high-latency conditions
src/scripts/start-marklogic.sh Updates curl timeouts, retry logic, and adds special handling for instance-admin endpoint
src/scripts/start-marklogic-rootless.sh Same timeout and retry improvements as main script for rootless containers

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Comment thread test/keywords.resource
Comment thread src/scripts/start-marklogic.sh Outdated
Comment thread src/scripts/start-marklogic-rootless.sh Outdated
Comment thread src/scripts/start-marklogic.sh
Comment thread src/scripts/start-marklogic-rootless.sh
@vitalykorolev vitalykorolev merged commit 0031554 into marklogic:develop Sep 8, 2025
1 check passed
@vitalykorolev vitalykorolev deleted the MLE-22761_docker-init-fix branch November 10, 2025 23:37
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.

3 participants