CondCore/CondDB/test/testConnectionPool.cpp accessing obsolete squid service

[DrDaveD]

CondCore/CondDB/test/testConnectionPool.cpp has a hardcoded frontier configuration, proxyconfigurl=http://cmst0frontier.cern.ch/t0.pac, and that is now obsolete and will soon be decomissioned. The new proxyconfigurl is http://grid-wpad/wpad.dat, but more importantly, why is that connect string hardcoded there at all? It should be using the local configuration from site-local-config.xml like everything else.

Please look at this at high priority, or these tests will start failing soon.

[cmsbuild]

A new Issue was created by @DrDaveD .

@davidlange6, @Dr15Jones, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[davidlange6]

assign db

…

On Apr 6, 2017, at 6:28 PM, cmsbuild ***@***.***> wrote: A new Issue was created by @DrDaveD . @davidlange6, @Dr15Jones, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here cms-sw/cmssw#13029 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[cmsbuild]

New categories assigned: db

@ggovi you have been requested to review this Pull request/Issue and eventually sign? Thanks

[DrDaveD]

@ggovi Please look at this with high priority. The t0.pac interface will soon be decomissioned and I assume all tests running this code will start breaking.

[ggovi]

@DrDaveD
Thanks for the warning. Addressed by
#18376
#18377
#18378
Concerning the need to have the full string in the test: here we validate the code parsing and decoding various connection string supported. The explicit frontier connection string, although discouraged, is supported.

[DrDaveD]

Thanks, @ggovi. Do you know if this test is run only on the latest code? Or is it also run on old code? Clearly it has to be run only at CERN since a CERN configuration is hard-coded.

[ggovi]

I found this test in 80X, 90X and 91X ( not in 71X ). Any other release cycle to check?

[DrDaveD]

It's good to hear that it doesn't go back to very old code. I just hope that people aren't using it for older released versions in those release cycles, but instead are only using it with the latest integration builds.

[ggovi]

+1

[cmsbuild]

This issue is fully signed and ready to be closed.