EvalAI icon indicating copy to clipboard operation
EvalAI copied to clipboard

Resolve DNS Failures & Normalize Shell Scripts in Docker Environment

Open wahid18-maqs opened this issue 8 months ago • 1 comments

Fixes:

Summary

This PR includes the following changes:

  • Adds DNS configuration (8.8.8.8, 1.1.1.1) for key Docker services (django-1, worker, worker_py3_8, nodejs, stats-exporter) in docker-compose.yml to resolve Temporary failure in name resolution.
  • Adds .gitattributes with text=auto to normalize line endings and prevent platform-specific diffs.
  • Adds base.txt to centralize and simplify Python dependency management across environments.

Motivation

  • Fixes:
    • [Errno -3] Temporary failure in name resolution
    • /usr/bin/env: ‘bash\r’: No such file or directory
    • /usr/bin/env: use -[v]S to pass options in shebang lines
  • Prevents CRLF-related script failures.
  • Improves cross-environment consistency and maintainability of dependencies.

Testing Done

Verified DNS resolution inside containers using:

python -c "import socket; print(socket.gethostbyname('google.com'))"

eval-dns

wahid18-maqs avatar Apr 19 '25 08:04 wahid18-maqs

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.69%. Comparing base (96968d6) to head (a3697f9). Report is 1164 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4604      +/-   ##
==========================================
- Coverage   72.93%   69.69%   -3.24%     
==========================================
  Files          83       21      -62     
  Lines        5368     3607    -1761     
==========================================
- Hits         3915     2514    -1401     
+ Misses       1453     1093     -360     

see 65 files with indirect coverage changes

see 65 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8ca8ade...a3697f9. Read the comment docs.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Apr 19 '25 09:04 codecov[bot]