Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sonar-project.properties
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@
sonar.projectKey=GPUSW_cuOpt_Nvidia-cuOpt_cuopt
sonar.projectName=NVIDIA cuOpt
sonar.projectVersion=1.0

sonar.host.url=https://sonar.nvidia.com
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Mar 31, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 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 10

Repository: 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 -20

Repository: 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/null

Repository: NVIDIA/cuopt

Length of output: 156


🏁 Script executed:

cat sonarqube/README.md

Repository: NVIDIA/cuopt

Length of output: 1459


🏁 Script executed:

# Check the full sonar-project.properties file to see all context
cat sonar-project.properties

Repository: NVIDIA/cuopt

Length of output: 410


🏁 Script executed:

# Search for any other references to sonar.nvidia.com or variations
rg "sonar\." -A 1 -B 1

Repository: NVIDIA/cuopt

Length of output: 662


🏁 Script executed:

# Check git history or comments for the hostname
git log --all --oneline --grep="sonar" | head -20

Repository: NVIDIA/cuopt

Length of output: 147


🏁 Script executed:

# Check the recent commit that updated the sonar host URL
git show ab53a50 --stat
git show ab53a50

Repository: 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.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is not available in public

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@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!

# Source code location
sonar.sources=.
2 changes: 1 addition & 1 deletion sonarqube/sonar-branches.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

# Main development branches
main
release/26.02
release/26.04

# Add release branches as needed
# release/v1.0
Expand Down
Loading