Udated docker-wait-for-dependencies to 26.1.0 #2220
Merged
+1
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WalkthroughThe Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom Pre-merge checks in the settings. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @docker-compose.yml:
- Line 251: The docker-compose image uses the unstable tag
drevops/docker-wait-for-dependencies:canary; verify that the intended stable tag
26.1.0 exists (or determine the correct stable version) and replace
drevops/docker-wait-for-dependencies:canary with the pinned tag (e.g.,
drevops/docker-wait-for-dependencies:26.1.0) in docker-compose.yml so builds are
deterministic, and validate by attempting a docker pull of the chosen tag.
📜 Review details
Configuration used: Repository UI
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (1)
docker-compose.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (11)
- GitHub Check: build (1)
- GitHub Check: build (0)
- GitHub Check: vortex-test-workflow (2)
- GitHub Check: vortex-test-installer (8.2)
- GitHub Check: vortex-test-workflow (0)
- GitHub Check: vortex-test-installer (8.4)
- GitHub Check: vortex-test-installer (8.3)
- GitHub Check: vortex-test-workflow (1)
- GitHub Check: vortex-test-workflow (3)
- GitHub Check: vortex-test-workflow (4)
- GitHub Check: vortex-test-common
This comment has been minimized.
This comment has been minimized.
Member
Author
|
Member
Author
|
Member
Author
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2220 +/- ##
==========================================
- Coverage 76.84% 76.20% -0.64%
==========================================
Files 112 105 -7
Lines 5842 5683 -159
Branches 44 0 -44
==========================================
- Hits 4489 4331 -158
+ Misses 1353 1352 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
AlexSkrypnyk
force-pushed
the
feature/update-docker-wait-for-deps-jan-26
branch
from
faa17af to
a0d4e50
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @docker-compose.yml:
- Line 251: The docker-compose service references a non-existent image tag
"drevops/docker-wait-for-dependencies:26.1.0"; update the image reference in
docker-compose.yml to a valid tag (e.g., the documented
"drevops/docker-wait-for-dependencies:23.12.0" or the latest available) by
replacing "drevops/docker-wait-for-dependencies:26.1.0" with the confirmed tag,
or pin to a specific digest after verifying on Docker Hub to ensure pulls won't
fail.
📜 Review details
Configuration used: Repository UI
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (1)
docker-compose.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (11)
- GitHub Check: build (0)
- GitHub Check: build (1)
- GitHub Check: vortex-test-workflow (1)
- GitHub Check: vortex-test-workflow (2)
- GitHub Check: vortex-test-workflow (4)
- GitHub Check: vortex-test-workflow (3)
- GitHub Check: vortex-test-workflow (0)
- GitHub Check: vortex-test-common
- GitHub Check: vortex-test-installer (8.3)
- GitHub Check: vortex-test-installer (8.2)
- GitHub Check: vortex-test-installer (8.4)
| # Helper container to wait for services to become available. | ||
| wait-for-dependencies: | ||
| image: drevops/docker-wait-for-dependencies:25.10.0 | ||
| image: drevops/docker-wait-for-dependencies:26.1.0 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Query Docker Hub API for drevops/docker-wait-for-dependencies available tags
echo "=== Querying Docker Hub for available tags ==="
curl -s "https://hub.docker.com/v2/repositories/drevops/docker-wait-for-dependencies/tags?page_size=100" | \
python3 << 'PYTHON_EOF'
import json
import sys
try:
data = json.load(sys.stdin)
if 'results' in data:
print(f"Total tags found: {len(data['results'])}")
# Look for specific versions
versions_of_interest = ['26.1.0', '25.10.0', '26.0.0', '26.1.1', '26.2.0']
found = {}
for result in data['results']:
name = result.get('name', '')
if name in versions_of_interest:
found[name] = result.get('last_updated', 'unknown')
if found:
print("\nVersions of interest found:")
for version, updated in sorted(found.items()):
print(f" {version} - last updated: {updated}")
else:
print("\nVersions of interest NOT found. Available tags (first 20):")
for i, result in enumerate(data['results'][:20]):
print(f" {result.get('name', 'unknown')}")
else:
print("No results found in API response")
except Exception as e:
print(f"Error: {e}")
PYTHON_EOFRepository: drevops/vortex
Length of output: 144
🌐 Web query:
drevops/docker-wait-for-dependencies 26.1.0 version Docker Hub
💡 Result:
I looked up the drevops/docker-wait-for-dependencies image on Docker Hub — the repository exists and shows a "latest" tag (example usage in the README references tag 23.12.0), but I couldn't find a tag named 26.1.0 on the repository page. [1][2]
Sources:
- drevops/docker-wait-for-dependencies (Docker Hub repository overview). [1]
- drevops/docker-wait-for-dependencies (Docker Hub tags view / repository page). [2]
Docker image tag 26.1.0 does not exist — this will cause build failures.
The tag 26.1.0 for drevops/docker-wait-for-dependencies is not available on Docker Hub. The repository exists, but this specific version cannot be found on the repository page. The documented version in the README is 23.12.0. Verify the correct available tag and update before merging, as builds will fail when attempting to pull a non-existent image.
🤖 Prompt for AI Agents
In @docker-compose.yml at line 251, The docker-compose service references a
non-existent image tag "drevops/docker-wait-for-dependencies:26.1.0"; update the
image reference in docker-compose.yml to a valid tag (e.g., the documented
"drevops/docker-wait-for-dependencies:23.12.0" or the latest available) by
replacing "drevops/docker-wait-for-dependencies:26.1.0" with the confirmed tag,
or pin to a specific digest after verifying on Docker Hub to ensure pulls won't
fail.
|
Member
Author
|
Member
Author
|
Member
Author
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.