Jim Bosch added a comment - 19/Jun/15 1:12 PM Changes on tickets/ DM-2949 of datarel: $ git --no-pager log --reverse LSST/master..tickets/DM-2949 commit 6e5f88858b38e7001cd27a46dfb8a702ffc46c8a Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Fri Jun 19 15:40:16 2015 -0400   Remove relic pipeline and execution code. This removes all code related to Stages, the old weekly productions, and the JobOffice, leaving only ingest and a few utility functions.   commit c82d5d93a4870580b3bb5ef1f601a83d60404622 Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Fri Jun 19 15:43:03 2015 -0400   Remove all ingest code that relies on lsst.ap, and the dependency on ap. Premise is that the code we'll want to start from when we get ingest working again is the code that doesn't use ap.   commit fb5cabc0f8c8d879b95b8bd4359c9963a47c73cb Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Fri Jun 19 15:45:21 2015 -0400   Re-enable unit tests for utility functions and ingest. Both of these tests were already working, and probably should have been enabled earlier; a comment in the SConscript suggests someone wanted to just disable all pex_harness-related tests, but got these as well. The ingest test does require access to a remote server, but I've verified that it skips the test and issues a warning when the server cannot be contacted, so I think it's safe. and lsst_distrib: $ git show LSST/master..tickets/DM-2949 commit 29f62650a8dd25d7528f4eca44013bab354c539d Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Fri Jun 19 15:55:44 2015 -0400   Remove dependency on testing_endToEnd, which is being retired.   diff --git a/ups/lsst_distrib.table b/ups/lsst_distrib.table index 05389da..60420cb 100644 --- a/ups/lsst_distrib.table +++ b/ups/lsst_distrib.table @@ -4,4 +4,3 @@ setupRequired(ctrl_execute) setupRequired(ctrl_platform_gordon) setupRequired(ctrl_platform_lsst) setupRequired(datarel) -setupRequired(testing_endtoend)

Jim Bosch added a comment - 19/Jun/15 1:39 PM Buildbot run successful: http://lsst-buildx.ncsa.illinois.edu:8010/builders/DM_stack/builds/7224

Kian-Tat Lim added a comment - 19/Jun/15 1:50 PM Does it make sense to change the name of what's now left after this ticket to daf_ingest and replace datarel with a new package that just has doc and ups subdirectories (and, I guess, an SConstruct )?

Kian-Tat Lim added a comment - 19/Jun/15 1:51 PM The removals all look appropriate to me, as does what is left behind. Sad to see some of that stuff go, particularly when we don't necessarily have good replacements for all of it yet...

Jim Bosch added a comment - 22/Jun/15 7:15 AM I don't think it makes sense to change the name yet, as we're also still using this package as a metapackage for the cross-linked Doxygen build (see DM-2948 ). That means we're still carrying more dependencies than the ingest code here needs, and we can't change the name without changing the buildbot scripts (which I can't do without help from SQuaRE, and they're busy).

Jim Bosch added a comment - 22/Jun/15 7:22 AM Merged to master.

Kian-Tat Lim added a comment - 22/Jun/15 7:23 AM Jim Bosch You missed the part about adding a new (nearly-empty) datarel that would serve as the metapackage for the cross-linked Doxygen build and the buildbot scripts.

Jim Bosch added a comment - 22/Jun/15 7:30 AM Oh, I did miss that part somehow, reading too fast - sorry about that. I'm still not sure it's worthwhile to do right now, as I think a better solution for the Doxygen/buildbot problem is to remove the need for a single top-level package. But we may want to do that first if Jacek Becla 's team starts working on updating the ingest code before DM-2948 is addressed.

John Swinbank added a comment - 23/Jun/15 7:10 AM Adding this to DM-1769 since it was necessary to unblock DM-1766 .