Use latest version of Nginx, 1.29.6

[xlamorlette-datadog]

Motivation

Following the delivery of Nginx Datadog 1.13.0, use, in System Tests, the latest Nginx version.

Changes

• Upgrade the used Nginx version from 1.29.5 to 1.29.6.
• In utils/scripts/load-binary.sh, remove deprecated get_circleci_artifact().

This PR works with DataDog/nginx-datadog#325.

Workflow

1. ⚠️ Create your PR as draft ⚠️
2. Work on you PR until the CI passes
3. Mark it as ready for review
   • Test logic is modified? -> Get a review from RFC owner.
   • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

• Anything but tests/ or manifests/ is modified ? I have the approval from R&P team
• A docker base image is modified?
  • the relevant build-XXX-image label is present
• A scenario is added, removed or renamed?
  • Get a review from R&P team

[github-actions]

CODEOWNERS have been resolved as:

utils/build/docker/cpp_nginx/nginx.Dockerfile                           @DataDog/system-tests-core
utils/scripts/load-binary.sh                                            @DataDog/system-tests-core

[datadog-prod-us1-4]

⚠️ Tests

✨ Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 1 Test failed

tests.debugger.test_debugger_inproduct_enablement.Test_Debugger_InProduct_Enablement_Exception_Replay.test_inproduct_enablement_exception_replay[poc] from system_tests_suite (Datadog) (Fix with Cursor)

AssertionError: Expected snapshots to be emitting after enabling exception replay
assert False
 +  where False = <tests.debugger.test_debugger_inproduct_enablement.Test_Debugger_InProduct_Enablement_Exception_Replay object at 0x7f36f37d8c80>.er_explicit_enabled

self = <tests.debugger.test_debugger_inproduct_enablement.Test_Debugger_InProduct_Enablement_Exception_Replay object at 0x7f36f37d8c80>

    def test_inproduct_enablement_exception_replay(self):
        self.assert_rc_state_not_error()
        self.assert_all_weblog_responses_ok(expected_code=500)
    
...

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ad8a376 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!}

[zacharycmontoya]

I can't seem to write a suggestion since you removed the line, but you should replace From circle ci with Github action artifact

[xlamorlette-datadog]

Well, I think this comment is actually useless (*), and one can see that other comments are wrong (*) and that several components are missing in this list.

(*): always prefer clear naming in code rather than a comment that can become out of sync with the code ;-)