[11u] RFR: 8222299: [TESTBUG] move hotspot container tests to hotspot/containers

[Severin Gehwolf]

On Mon, 2019-08-19 at 19:32 +0200, Aleksey Shipilev wrote:
> On 8/19/19 5:20 PM, Severin Gehwolf wrote:
> > JDK-8221894: http://mail.openjdk.java.net/pipermail/jdk-updates-dev/2019-August/001684.html
> > JDK-8219997: http://mail.openjdk.java.net/pipermail/jdk-updates-dev/2019-August/001681.html
> > 
> > I'm not sure what you mean (perhaps even a moot point now?). 8219997
> > isn't (yet) in OpenJDK 11u, thus JDK-8222299, could not possibly move
> > the files somewhere else. That was true for webrev 02. webrev 03
> > (latest) changed that by depending on JDK-8219997 backport. Perhaps I'm
> > missing something?

> I understand this is the order the changes went in jdk/jdk:
>   8219997 -- adds 2 new files to runtime/containers/docker
>   8222299 -- moves all files from runtime/containers/docker to containers/docker
> 
> Stands to reason that if you do 8219997 first, you would get files in their old locations, and then
> 8222299 moves them to the new ones. Then, 11u history and jdk/jdk history would agree: in both
> histories, 2 new files would be added by 8219997.

Yes, that's my understanding as well.

> Your current webrev says:
>  test/hotspot/jtreg/containers/docker/JfrReporter.java (was
> test/hotspot/jtreg/runtime/containers/docker/JfrReporter.java)
>  test/hotspot/jtreg/containers/docker/TestJFREvents.java (was
> test/hotspot/jtreg/runtime/containers/docker/TestJFREvents.java)

Current webrev (03) has 8219997 as a pre-requisite, so yes I have that
in my queue.

> ...so the 8219997 changeset *is* in your patch queue, it just does not show up in webrev file
> history for JfrReporter.java and TestJFREvents.java? That is what confused me.

OK. Sorry if that wasn't clear. Yes, webrev 03, has two pre-requisites: 
8219997 and 8221894.

Thanks,
Severin