Update sonarcube host link to Nvidia#1015
Conversation
📝 WalkthroughWalkthroughUpdates SonarQube configuration by adding a host URL property to the main configuration file and updating the monitored release branch from version 26.02 to 26.04 in the branch list. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@sonar-project.properties`:
- Line 8: The sonar.host.url property value "https://sonar.nvidia.com" is
unreachable; confirm the correct SonarQube hostname with the cuOpt team (per
sonarqube/README.md) and update the sonar.host.url property to the verified host
(e.g., sonarqube.nvidia.com or the internal domain). Alternatively make
sonar.host.url configurable via CI environment variables and reference that
variable in the properties file so CI can override the host; ensure the final
value resolves via DNS and is accessible from the CI runners before merging.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: abff35b2-4994-4ba9-a456-97117f6c57c7
📒 Files selected for processing (2)
sonar-project.propertiessonarqube/sonar-branches.txt
| sonar.projectName=NVIDIA cuOpt | ||
| sonar.projectVersion=1.0 | ||
|
|
||
| sonar.host.url=https://sonar.nvidia.com |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify the SonarQube host URL is accessible
echo "Testing accessibility of SonarQube host..."
curl -s -o /dev/null -w "HTTP Status: %{http_code}\n" https://sonar.nvidia.com
echo -e "\nChecking if URL redirects or requires authentication..."
curl -I https://sonar.nvidia.com 2>&1 | head -n 10Repository: NVIDIA/cuopt
Length of output: 460
🏁 Script executed:
# Search for other SonarQube references in the repository
rg -i "sonar" --type-list | head -20
rg -i "sonar" -g "*.properties" -g "*.yml" -g "*.yaml" -g "*.json" -g "*.md" -g "*.sh"Repository: NVIDIA/cuopt
Length of output: 3111
🏁 Script executed:
# Check for CI/CD configuration files
fd -e "yml" -e "yaml" -e "json" -e "sh" | rg -i "github|gitlab|jenkins|ci|workflow" | head -20Repository: NVIDIA/cuopt
Length of output: 542
🏁 Script executed:
# Look for any documentation or README files
fd -e "md" -e "txt" | xargs rg -l -i "sonar" 2>/dev/nullRepository: NVIDIA/cuopt
Length of output: 156
🏁 Script executed:
cat sonarqube/README.mdRepository: NVIDIA/cuopt
Length of output: 1459
🏁 Script executed:
# Check the full sonar-project.properties file to see all context
cat sonar-project.propertiesRepository: NVIDIA/cuopt
Length of output: 410
🏁 Script executed:
# Search for any other references to sonar.nvidia.com or variations
rg "sonar\." -A 1 -B 1Repository: NVIDIA/cuopt
Length of output: 662
🏁 Script executed:
# Check git history or comments for the hostname
git log --all --oneline --grep="sonar" | head -20Repository: NVIDIA/cuopt
Length of output: 147
🏁 Script executed:
# Check the recent commit that updated the sonar host URL
git show ab53a50 --stat
git show ab53a50Repository: NVIDIA/cuopt
Length of output: 50371
🏁 Script executed:
# Check the previous version to see what URL was used before
git show 5d2ab5a:sonar-project.properties 2>/dev/null || echo "File may not have existed"Repository: NVIDIA/cuopt
Length of output: 371
Critical: The SonarQube host URL fails DNS resolution.
The hostname sonar.nvidia.com cannot be resolved. Testing shows:
curl: (6) Could not resolve host: sonar.nvidia.com
HTTP Status: 000
When sonar-scanner executes during CI/CD with this configuration, it will fail immediately with a connection error. Verify the correct SonarQube instance hostname with the cuOpt team (as referenced in sonarqube/README.md). Possible alternatives include sonarqube.nvidia.com, an internal-only domain, or a different host configuration entirely.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@sonar-project.properties` at line 8, The sonar.host.url property value
"https://sonar.nvidia.com" is unreachable; confirm the correct SonarQube
hostname with the cuOpt team (per sonarqube/README.md) and update the
sonar.host.url property to the verified host (e.g., sonarqube.nvidia.com or the
internal domain). Alternatively make sonar.host.url configurable via CI
environment variables and reference that variable in the properties file so CI
can override the host; ensure the final value resolves via DNS and is accessible
from the CI runners before merging.
There was a problem hiding this comment.
This is not available in public
There was a problem hiding this comment.
@rgsl888prabhu, understood! The sonar.nvidia.com hostname is an internal NVIDIA domain, not reachable from public networks. My DNS resolution check was run from a public sandbox environment, which naturally wouldn't have access to NVIDIA's internal infrastructure. The URL is correct and will resolve fine from NVIDIA's CI runners. Apologies for the false positive!
rgsl888prabhu
added
bug
|
/merge |
2 participants
Description
Update sonarcube host link to Nvidia
Checklist