Comment 3 for bug 2051695

Revision history for this message
Pushkar Kulkarni (pushkarnk) wrote :

The new failure path starts in get_remote_storage_config_for_logging() where storage::get_redacted_url_str_for_logging() is now invoked.

storage()::get_remote_storage_config_for_logging()
 |
 storage::get_redacted_url_str_for_logging()
  |
  Url::str()
   |
    Url::build_url()
     |
     throw Url::build_error

Though the Url::build_error is legit, whether it should be caught is a question. Or rather, is not catching it an upstream bug?

Interestingly, the autopkgtests pass, but still fail. This is because "Aborted (core dumped)" starts appearing on stderr.

---
820s PASSED <<<<<<<<<
821s autopkgtest [19:49:21]: test run-testsuites: -----------------------]
821s autopkgtest [19:49:21]: test run-testsuites: - - - - - - - - - - results - - - - - - - - - -
821s run-testsuites FAIL stderr: /tmp/autopkgtest.b0jEpT/build.bnS/src/test/suites/remote_url.bash: line 5: 23875 Aborted (core dumped) $CCACHE_COMPILE -c test.c 2> stderr.log <<<<<<<<<
---