* [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts @ 2021-02-28 22:43 Philippe Mathieu-Daudé 2021-03-01 13:30 ` Wainer dos Santos Moschetta 0 siblings, 1 reply; 6+ messages in thread From: Philippe Mathieu-Daudé @ 2021-02-28 22:43 UTC (permalink / raw) To: qemu-devel Cc: Philippe Mathieu-Daudé, Wainer dos Santos Moschetta, Pavel Dovgalyuk, Cleber Rosa, Paolo Bonzini, Philippe Mathieu-Daudé Having artifacts stored in personal namespace is not ideal, as these might get closed and disappear. Use the original URL where these artifacts could be found. For more references: https://fedoraproject.org/wiki/Architectures/MIPS/2015Bootstrap/mips64el Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> --- tests/acceptance/boot_linux_console.py | 4 ++-- tests/acceptance/replay_kernel.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py index eb012867997..138bd8cb7d3 100644 --- a/tests/acceptance/boot_linux_console.py +++ b/tests/acceptance/boot_linux_console.py @@ -239,8 +239,8 @@ def test_mips64el_malta_5KEc_cpio(self): :avocado: tags=machine:malta :avocado: tags=endian:little """ - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' - 'raw/9ad2df38/mips/malta/mips64el/' + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' + 'mips64el/images/20150601/' 'vmlinux-3.19.3.mtoman.20150408') kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) diff --git a/tests/acceptance/replay_kernel.py b/tests/acceptance/replay_kernel.py index c1cb8624683..efdd4233bc2 100644 --- a/tests/acceptance/replay_kernel.py +++ b/tests/acceptance/replay_kernel.py @@ -397,8 +397,8 @@ def test_mips64el_malta_5KEc_cpio(self): :avocado: tags=endian:little :avocado: tags=slowness:high """ - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' - 'raw/9ad2df38/mips/malta/mips64el/' + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' + 'mips64el/images/20150601/' 'vmlinux-3.19.3.mtoman.20150408') kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) -- 2.26.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts 2021-02-28 22:43 [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts Philippe Mathieu-Daudé @ 2021-03-01 13:30 ` Wainer dos Santos Moschetta 2021-03-01 13:56 ` Philippe Mathieu-Daudé 0 siblings, 1 reply; 6+ messages in thread From: Wainer dos Santos Moschetta @ 2021-03-01 13:30 UTC (permalink / raw) To: Philippe Mathieu-Daudé, qemu-devel Cc: Paolo Bonzini, Philippe Mathieu-Daudé, Pavel Dovgalyuk, Cleber Rosa Hi, On 2/28/21 7:43 PM, Philippe Mathieu-Daudé wrote: > Having artifacts stored in personal namespace is not ideal, > as these might get closed and disappear. Use the original URL > where these artifacts could be found. > > For more references: > https://fedoraproject.org/wiki/Architectures/MIPS/2015Bootstrap/mips64el > > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > tests/acceptance/boot_linux_console.py | 4 ++-- > tests/acceptance/replay_kernel.py | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py > index eb012867997..138bd8cb7d3 100644 > --- a/tests/acceptance/boot_linux_console.py > +++ b/tests/acceptance/boot_linux_console.py > @@ -239,8 +239,8 @@ def test_mips64el_malta_5KEc_cpio(self): > :avocado: tags=machine:malta > :avocado: tags=endian:little > """ > - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > - 'raw/9ad2df38/mips/malta/mips64el/' > + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' > + 'mips64el/images/20150601/' > 'vmlinux-3.19.3.mtoman.20150408') http://mipsfedora.imgtec.com/development/22/mips64el/images/20150601/vmlinux-3.19.3.mtoman.20150408 did not get resolve here. Could you please double-check the url is correct? Thanks! Wainer > kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) > diff --git a/tests/acceptance/replay_kernel.py b/tests/acceptance/replay_kernel.py > index c1cb8624683..efdd4233bc2 100644 > --- a/tests/acceptance/replay_kernel.py > +++ b/tests/acceptance/replay_kernel.py > @@ -397,8 +397,8 @@ def test_mips64el_malta_5KEc_cpio(self): > :avocado: tags=endian:little > :avocado: tags=slowness:high > """ > - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > - 'raw/9ad2df38/mips/malta/mips64el/' > + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' > + 'mips64el/images/20150601/' > 'vmlinux-3.19.3.mtoman.20150408') > kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts 2021-03-01 13:30 ` Wainer dos Santos Moschetta @ 2021-03-01 13:56 ` Philippe Mathieu-Daudé 2021-03-02 15:27 ` Willian Rampazzo 0 siblings, 1 reply; 6+ messages in thread From: Philippe Mathieu-Daudé @ 2021-03-01 13:56 UTC (permalink / raw) To: Wainer dos Santos Moschetta, qemu-devel Cc: Paolo Bonzini, Philippe Mathieu-Daudé, Pavel Dovgalyuk, Cleber Rosa On 3/1/21 2:30 PM, Wainer dos Santos Moschetta wrote: > Hi, > > On 2/28/21 7:43 PM, Philippe Mathieu-Daudé wrote: >> Having artifacts stored in personal namespace is not ideal, >> as these might get closed and disappear. Use the original URL >> where these artifacts could be found. >> >> For more references: >> https://fedoraproject.org/wiki/Architectures/MIPS/2015Bootstrap/mips64el >> >> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> >> --- >> tests/acceptance/boot_linux_console.py | 4 ++-- >> tests/acceptance/replay_kernel.py | 4 ++-- >> 2 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/tests/acceptance/boot_linux_console.py >> b/tests/acceptance/boot_linux_console.py >> index eb012867997..138bd8cb7d3 100644 >> --- a/tests/acceptance/boot_linux_console.py >> +++ b/tests/acceptance/boot_linux_console.py >> @@ -239,8 +239,8 @@ def test_mips64el_malta_5KEc_cpio(self): >> :avocado: tags=machine:malta >> :avocado: tags=endian:little >> """ >> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' >> - 'raw/9ad2df38/mips/malta/mips64el/' >> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' >> + 'mips64el/images/20150601/' >> 'vmlinux-3.19.3.mtoman.20150408') > > http://mipsfedora.imgtec.com/development/22/mips64el/images/20150601/vmlinux-3.19.3.mtoman.20150408 > did not get resolve here. Could you please double-check the url is correct? Yes it is correct, the domain is dead. It was working when I wrote the test. Should I remove the test instead? >> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' >> kernel_path = self.fetch_asset(kernel_url, >> asset_hash=kernel_hash) >> diff --git a/tests/acceptance/replay_kernel.py >> b/tests/acceptance/replay_kernel.py >> index c1cb8624683..efdd4233bc2 100644 >> --- a/tests/acceptance/replay_kernel.py >> +++ b/tests/acceptance/replay_kernel.py >> @@ -397,8 +397,8 @@ def test_mips64el_malta_5KEc_cpio(self): >> :avocado: tags=endian:little >> :avocado: tags=slowness:high >> """ >> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' >> - 'raw/9ad2df38/mips/malta/mips64el/' >> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' >> + 'mips64el/images/20150601/' >> 'vmlinux-3.19.3.mtoman.20150408') >> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' >> kernel_path = self.fetch_asset(kernel_url, >> asset_hash=kernel_hash) > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts 2021-03-01 13:56 ` Philippe Mathieu-Daudé @ 2021-03-02 15:27 ` Willian Rampazzo 2021-03-02 15:45 ` Philippe Mathieu-Daudé 0 siblings, 1 reply; 6+ messages in thread From: Willian Rampazzo @ 2021-03-02 15:27 UTC (permalink / raw) To: Philippe Mathieu-Daudé Cc: qemu-devel, Wainer dos Santos Moschetta, Pavel Dovgalyuk, Cleber Rosa, Paolo Bonzini, Philippe Mathieu-Daudé Hi Phil, On Mon, Mar 1, 2021 at 11:11 AM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: > > On 3/1/21 2:30 PM, Wainer dos Santos Moschetta wrote: > > Hi, > > > > On 2/28/21 7:43 PM, Philippe Mathieu-Daudé wrote: > >> Having artifacts stored in personal namespace is not ideal, > >> as these might get closed and disappear. Use the original URL > >> where these artifacts could be found. > >> > >> For more references: > >> https://fedoraproject.org/wiki/Architectures/MIPS/2015Bootstrap/mips64el > >> > >> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > >> --- > >> tests/acceptance/boot_linux_console.py | 4 ++-- > >> tests/acceptance/replay_kernel.py | 4 ++-- > >> 2 files changed, 4 insertions(+), 4 deletions(-) > >> > >> diff --git a/tests/acceptance/boot_linux_console.py > >> b/tests/acceptance/boot_linux_console.py > >> index eb012867997..138bd8cb7d3 100644 > >> --- a/tests/acceptance/boot_linux_console.py > >> +++ b/tests/acceptance/boot_linux_console.py > >> @@ -239,8 +239,8 @@ def test_mips64el_malta_5KEc_cpio(self): > >> :avocado: tags=machine:malta > >> :avocado: tags=endian:little > >> """ > >> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > >> - 'raw/9ad2df38/mips/malta/mips64el/' > >> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' > >> + 'mips64el/images/20150601/' > >> 'vmlinux-3.19.3.mtoman.20150408') > > > > http://mipsfedora.imgtec.com/development/22/mips64el/images/20150601/vmlinux-3.19.3.mtoman.20150408 > > did not get resolve here. Could you please double-check the url is correct? > > Yes it is correct, the domain is dead. It was working when I wrote > the test. Should I remove the test instead? > This test is already skipped by default because of the AVOCADO_ALLOW_UNTRUSTED_CODE flag. Even if the flag is set to allow the test to run, it will be skipped because the cache lacks the asset. Instead of removing the test, my suggestion is to add information to allow people to run it locally, like: diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py index eb01286799..23efc18a99 100644 --- a/tests/acceptance/boot_linux_console.py +++ b/tests/acceptance/boot_linux_console.py @@ -239,11 +239,13 @@ def test_mips64el_malta_5KEc_cpio(self): :avocado: tags=machine:malta :avocado: tags=endian:little """ - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' - 'raw/9ad2df38/mips/malta/mips64el/' - 'vmlinux-3.19.3.mtoman.20150408') + # to run this test locally, download the following asset and register + # it manually to the avocado cache with: + # $(BUILD_DIR)/tests/venv/bin/avocado assets register \ + # vmlinux-3.19.3.mtoman.20150408 <path to the file> + kernel_name = 'vmlinux-3.19.3.mtoman.20150408' kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' - kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) + kernel_path = self.fetch_asset(kernel_name, asset_hash=kernel_hash) initrd_url = ('https://github.com/groeck/linux-build-test/' 'raw/8584a59e/rootfs/' 'mipsel64/rootfs.mipsel64r1.cpio.gz') diff --git a/tests/acceptance/replay_kernel.py b/tests/acceptance/replay_kernel.py index c1cb862468..180e915264 100644 --- a/tests/acceptance/replay_kernel.py +++ b/tests/acceptance/replay_kernel.py @@ -397,11 +397,13 @@ def test_mips64el_malta_5KEc_cpio(self): :avocado: tags=endian:little :avocado: tags=slowness:high """ - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' - 'raw/9ad2df38/mips/malta/mips64el/' - 'vmlinux-3.19.3.mtoman.20150408') + # to run this test locally, download the following asset and register + # it manually to the avocado cache with: + # $(BUILD_DIR)/tests/venv/bin/avocado assets register \ + # vmlinux-3.19.3.mtoman.20150408 <path to the file> + kernel_name = 'vmlinux-3.19.3.mtoman.20150408' kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' - kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) + kernel_path = self.fetch_asset(kernel_name, asset_hash=kernel_hash) initrd_url = ('https://github.com/groeck/linux-build-test/' 'raw/8584a59e/rootfs/' 'mipsel64/rootfs.mipsel64r1.cpio.gz') What do you think? ps. WARNING, above code not tested. > >> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > >> kernel_path = self.fetch_asset(kernel_url, > >> asset_hash=kernel_hash) > >> diff --git a/tests/acceptance/replay_kernel.py > >> b/tests/acceptance/replay_kernel.py > >> index c1cb8624683..efdd4233bc2 100644 > >> --- a/tests/acceptance/replay_kernel.py > >> +++ b/tests/acceptance/replay_kernel.py > >> @@ -397,8 +397,8 @@ def test_mips64el_malta_5KEc_cpio(self): > >> :avocado: tags=endian:little > >> :avocado: tags=slowness:high > >> """ > >> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > >> - 'raw/9ad2df38/mips/malta/mips64el/' > >> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' > >> + 'mips64el/images/20150601/' > >> 'vmlinux-3.19.3.mtoman.20150408') > >> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > >> kernel_path = self.fetch_asset(kernel_url, > >> asset_hash=kernel_hash) > > > ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts 2021-03-02 15:27 ` Willian Rampazzo @ 2021-03-02 15:45 ` Philippe Mathieu-Daudé 2021-03-03 18:52 ` Willian Rampazzo 0 siblings, 1 reply; 6+ messages in thread From: Philippe Mathieu-Daudé @ 2021-03-02 15:45 UTC (permalink / raw) To: Willian Rampazzo Cc: qemu-devel, Wainer dos Santos Moschetta, Pavel Dovgalyuk, Cleber Rosa, Paolo Bonzini, Philippe Mathieu-Daudé On 3/2/21 4:27 PM, Willian Rampazzo wrote: > Hi Phil, > > On Mon, Mar 1, 2021 at 11:11 AM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: >> >> On 3/1/21 2:30 PM, Wainer dos Santos Moschetta wrote: >>> Hi, >>> >>> On 2/28/21 7:43 PM, Philippe Mathieu-Daudé wrote: >>>> Having artifacts stored in personal namespace is not ideal, >>>> as these might get closed and disappear. Use the original URL >>>> where these artifacts could be found. >>>> >>>> For more references: >>>> https://fedoraproject.org/wiki/Architectures/MIPS/2015Bootstrap/mips64el >>>> >>>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> >>>> --- >>>> tests/acceptance/boot_linux_console.py | 4 ++-- >>>> tests/acceptance/replay_kernel.py | 4 ++-- >>>> 2 files changed, 4 insertions(+), 4 deletions(-) >>>> >>>> diff --git a/tests/acceptance/boot_linux_console.py >>>> b/tests/acceptance/boot_linux_console.py >>>> index eb012867997..138bd8cb7d3 100644 >>>> --- a/tests/acceptance/boot_linux_console.py >>>> +++ b/tests/acceptance/boot_linux_console.py >>>> @@ -239,8 +239,8 @@ def test_mips64el_malta_5KEc_cpio(self): >>>> :avocado: tags=machine:malta >>>> :avocado: tags=endian:little >>>> """ >>>> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' >>>> - 'raw/9ad2df38/mips/malta/mips64el/' >>>> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' >>>> + 'mips64el/images/20150601/' >>>> 'vmlinux-3.19.3.mtoman.20150408') >>> >>> http://mipsfedora.imgtec.com/development/22/mips64el/images/20150601/vmlinux-3.19.3.mtoman.20150408 >>> did not get resolve here. Could you please double-check the url is correct? >> >> Yes it is correct, the domain is dead. It was working when I wrote >> the test. Should I remove the test instead? >> > > This test is already skipped by default because of the > AVOCADO_ALLOW_UNTRUSTED_CODE flag. QEMU's .gitlab-ci.yml does (see acceptance_definition): before_script: - mkdir -p ~/.config/avocado - echo "[datadir.paths]" > ~/.config/avocado/avocado.conf - echo "cache_dirs = ['${CI_PROJECT_DIR}/avocado-cache']" >> ~/.config/avocado/avocado.conf - echo -e '[job.output.testlogs]\nstatuses = ["FAIL", "INTERRUPT"]' >> ~/.config/avocado/avocado.conf - if [ -d ${CI_PROJECT_DIR}/avocado-cache ]; then du -chs ${CI_PROJECT_DIR}/avocado-cache ; fi - export AVOCADO_ALLOW_UNTRUSTED_CODE=1 > Even if the flag is set to allow > the test to run, it will be skipped because the cache lacks the asset. Same file, we save the cache between runs: cache: key: "${CI_JOB_NAME}-cache" paths: - ${CI_PROJECT_DIR}/avocado-cache > Instead of removing the test, my suggestion is to add information to > allow people to run it locally, like: > > diff --git a/tests/acceptance/boot_linux_console.py > b/tests/acceptance/boot_linux_console.py > index eb01286799..23efc18a99 100644 > --- a/tests/acceptance/boot_linux_console.py > +++ b/tests/acceptance/boot_linux_console.py > @@ -239,11 +239,13 @@ def test_mips64el_malta_5KEc_cpio(self): > :avocado: tags=machine:malta > :avocado: tags=endian:little > """ > - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > - 'raw/9ad2df38/mips/malta/mips64el/' > - 'vmlinux-3.19.3.mtoman.20150408') > + # to run this test locally, download the following asset and register > + # it manually to the avocado cache with: > + # $(BUILD_DIR)/tests/venv/bin/avocado assets register \ > + # vmlinux-3.19.3.mtoman.20150408 <path to the file> > + kernel_name = 'vmlinux-3.19.3.mtoman.20150408' In this particular case it could work because the name is not generic, but what about the generic names ("vmlinux" for example)? Can't we register assets using any string to hash? > kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > - kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) > + kernel_path = self.fetch_asset(kernel_name, asset_hash=kernel_hash) > initrd_url = ('https://github.com/groeck/linux-build-test/' > 'raw/8584a59e/rootfs/' > 'mipsel64/rootfs.mipsel64r1.cpio.gz') > diff --git a/tests/acceptance/replay_kernel.py > b/tests/acceptance/replay_kernel.py > index c1cb862468..180e915264 100644 > --- a/tests/acceptance/replay_kernel.py > +++ b/tests/acceptance/replay_kernel.py > @@ -397,11 +397,13 @@ def test_mips64el_malta_5KEc_cpio(self): > :avocado: tags=endian:little > :avocado: tags=slowness:high > """ > - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > - 'raw/9ad2df38/mips/malta/mips64el/' > - 'vmlinux-3.19.3.mtoman.20150408') > + # to run this test locally, download the following asset and register > + # it manually to the avocado cache with: > + # $(BUILD_DIR)/tests/venv/bin/avocado assets register \ > + # vmlinux-3.19.3.mtoman.20150408 <path to the file> > + kernel_name = 'vmlinux-3.19.3.mtoman.20150408' > kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > - kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) > + kernel_path = self.fetch_asset(kernel_name, asset_hash=kernel_hash) > initrd_url = ('https://github.com/groeck/linux-build-test/' > 'raw/8584a59e/rootfs/' > 'mipsel64/rootfs.mipsel64r1.cpio.gz') > > What do you think? > > ps. WARNING, above code not tested. > >>>> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' >>>> kernel_path = self.fetch_asset(kernel_url, >>>> asset_hash=kernel_hash) >>>> diff --git a/tests/acceptance/replay_kernel.py >>>> b/tests/acceptance/replay_kernel.py >>>> index c1cb8624683..efdd4233bc2 100644 >>>> --- a/tests/acceptance/replay_kernel.py >>>> +++ b/tests/acceptance/replay_kernel.py >>>> @@ -397,8 +397,8 @@ def test_mips64el_malta_5KEc_cpio(self): >>>> :avocado: tags=endian:little >>>> :avocado: tags=slowness:high >>>> """ >>>> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' >>>> - 'raw/9ad2df38/mips/malta/mips64el/' >>>> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' >>>> + 'mips64el/images/20150601/' >>>> 'vmlinux-3.19.3.mtoman.20150408') >>>> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' >>>> kernel_path = self.fetch_asset(kernel_url, >>>> asset_hash=kernel_hash) >>> >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts 2021-03-02 15:45 ` Philippe Mathieu-Daudé @ 2021-03-03 18:52 ` Willian Rampazzo 0 siblings, 0 replies; 6+ messages in thread From: Willian Rampazzo @ 2021-03-03 18:52 UTC (permalink / raw) To: Philippe Mathieu-Daudé Cc: Pavel Dovgalyuk, Paolo Bonzini, qemu-devel, Wainer dos Santos Moschetta, Cleber Rosa Hi Phil, On Tue, Mar 2, 2021 at 12:45 PM Philippe Mathieu-Daudé <philmd@redhat.com> wrote: > > On 3/2/21 4:27 PM, Willian Rampazzo wrote: > > Hi Phil, > > > > On Mon, Mar 1, 2021 at 11:11 AM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: > >> > >> On 3/1/21 2:30 PM, Wainer dos Santos Moschetta wrote: > >>> Hi, > >>> > >>> On 2/28/21 7:43 PM, Philippe Mathieu-Daudé wrote: > >>>> Having artifacts stored in personal namespace is not ideal, > >>>> as these might get closed and disappear. Use the original URL > >>>> where these artifacts could be found. > >>>> > >>>> For more references: > >>>> https://fedoraproject.org/wiki/Architectures/MIPS/2015Bootstrap/mips64el > >>>> > >>>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > >>>> --- > >>>> tests/acceptance/boot_linux_console.py | 4 ++-- > >>>> tests/acceptance/replay_kernel.py | 4 ++-- > >>>> 2 files changed, 4 insertions(+), 4 deletions(-) > >>>> > >>>> diff --git a/tests/acceptance/boot_linux_console.py > >>>> b/tests/acceptance/boot_linux_console.py > >>>> index eb012867997..138bd8cb7d3 100644 > >>>> --- a/tests/acceptance/boot_linux_console.py > >>>> +++ b/tests/acceptance/boot_linux_console.py > >>>> @@ -239,8 +239,8 @@ def test_mips64el_malta_5KEc_cpio(self): > >>>> :avocado: tags=machine:malta > >>>> :avocado: tags=endian:little > >>>> """ > >>>> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > >>>> - 'raw/9ad2df38/mips/malta/mips64el/' > >>>> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' > >>>> + 'mips64el/images/20150601/' > >>>> 'vmlinux-3.19.3.mtoman.20150408') > >>> > >>> http://mipsfedora.imgtec.com/development/22/mips64el/images/20150601/vmlinux-3.19.3.mtoman.20150408 > >>> did not get resolve here. Could you please double-check the url is correct? > >> > >> Yes it is correct, the domain is dead. It was working when I wrote > >> the test. Should I remove the test instead? > >> > > > > This test is already skipped by default because of the > > AVOCADO_ALLOW_UNTRUSTED_CODE flag. > > QEMU's .gitlab-ci.yml does (see acceptance_definition): > > before_script: > - mkdir -p ~/.config/avocado > - echo "[datadir.paths]" > ~/.config/avocado/avocado.conf > - echo "cache_dirs = ['${CI_PROJECT_DIR}/avocado-cache']" > >> ~/.config/avocado/avocado.conf > - echo -e '[job.output.testlogs]\nstatuses = ["FAIL", "INTERRUPT"]' > >> ~/.config/avocado/avocado.conf > - if [ -d ${CI_PROJECT_DIR}/avocado-cache ]; then > du -chs ${CI_PROJECT_DIR}/avocado-cache ; > fi > - export AVOCADO_ALLOW_UNTRUSTED_CODE=1 > This is true when running the test in the CI. What happens if I try to run this test locally? It will be skipped unless I set AVOCADO_ALLOW_UNTRUSTED_CODE=1. > > Even if the flag is set to allow > > the test to run, it will be skipped because the cache lacks the asset. > > Same file, we save the cache between runs: > > cache: > key: "${CI_JOB_NAME}-cache" > paths: > - ${CI_PROJECT_DIR}/avocado-cache > In the CI environment, the test will execute because the asset is already cached. Again, what happens if we merge this code and I try to run it locally? the test will be skipped because I don't have the asset. The QEMU GitLab CI won't be affected by an invalid URL and will run the test until the cache is cleared. If someone tries to run the pipeline on a new instance of a GitLab CI, the test will be skipped. With my suggestion, developers will have the chance to run the test locally if they have the correct file and register it to their local cache manually. > > Instead of removing the test, my suggestion is to add information to > > allow people to run it locally, like: > > > > diff --git a/tests/acceptance/boot_linux_console.py > > b/tests/acceptance/boot_linux_console.py > > index eb01286799..23efc18a99 100644 > > --- a/tests/acceptance/boot_linux_console.py > > +++ b/tests/acceptance/boot_linux_console.py > > @@ -239,11 +239,13 @@ def test_mips64el_malta_5KEc_cpio(self): > > :avocado: tags=machine:malta > > :avocado: tags=endian:little > > """ > > - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > > - 'raw/9ad2df38/mips/malta/mips64el/' > > - 'vmlinux-3.19.3.mtoman.20150408') > > + # to run this test locally, download the following asset and register > > + # it manually to the avocado cache with: > > + # $(BUILD_DIR)/tests/venv/bin/avocado assets register \ > > + # vmlinux-3.19.3.mtoman.20150408 <path to the file> > > + kernel_name = 'vmlinux-3.19.3.mtoman.20150408' > > In this particular case it could work because the name is not generic, > but what about the generic names ("vmlinux" for example)? > > Can't we register assets using any string to hash? > Right now it is not supported, but I opened an issue to discuss it here: https://github.com/avocado-framework/avocado/issues/4435 > > kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > > - kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) > > + kernel_path = self.fetch_asset(kernel_name, asset_hash=kernel_hash) > > initrd_url = ('https://github.com/groeck/linux-build-test/' > > 'raw/8584a59e/rootfs/' > > 'mipsel64/rootfs.mipsel64r1.cpio.gz') > > diff --git a/tests/acceptance/replay_kernel.py > > b/tests/acceptance/replay_kernel.py > > index c1cb862468..180e915264 100644 > > --- a/tests/acceptance/replay_kernel.py > > +++ b/tests/acceptance/replay_kernel.py > > @@ -397,11 +397,13 @@ def test_mips64el_malta_5KEc_cpio(self): > > :avocado: tags=endian:little > > :avocado: tags=slowness:high > > """ > > - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > > - 'raw/9ad2df38/mips/malta/mips64el/' > > - 'vmlinux-3.19.3.mtoman.20150408') > > + # to run this test locally, download the following asset and register > > + # it manually to the avocado cache with: > > + # $(BUILD_DIR)/tests/venv/bin/avocado assets register \ > > + # vmlinux-3.19.3.mtoman.20150408 <path to the file> > > + kernel_name = 'vmlinux-3.19.3.mtoman.20150408' > > kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > > - kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash) > > + kernel_path = self.fetch_asset(kernel_name, asset_hash=kernel_hash) > > initrd_url = ('https://github.com/groeck/linux-build-test/' > > 'raw/8584a59e/rootfs/' > > 'mipsel64/rootfs.mipsel64r1.cpio.gz') > > > > What do you think? > > > > ps. WARNING, above code not tested. > > > >>>> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > >>>> kernel_path = self.fetch_asset(kernel_url, > >>>> asset_hash=kernel_hash) > >>>> diff --git a/tests/acceptance/replay_kernel.py > >>>> b/tests/acceptance/replay_kernel.py > >>>> index c1cb8624683..efdd4233bc2 100644 > >>>> --- a/tests/acceptance/replay_kernel.py > >>>> +++ b/tests/acceptance/replay_kernel.py > >>>> @@ -397,8 +397,8 @@ def test_mips64el_malta_5KEc_cpio(self): > >>>> :avocado: tags=endian:little > >>>> :avocado: tags=slowness:high > >>>> """ > >>>> - kernel_url = ('https://github.com/philmd/qemu-testing-blob/' > >>>> - 'raw/9ad2df38/mips/malta/mips64el/' > >>>> + kernel_url = ('http://mipsfedora.imgtec.com/development/22/' > >>>> + 'mips64el/images/20150601/' > >>>> 'vmlinux-3.19.3.mtoman.20150408') > >>>> kernel_hash = '00d1d268fb9f7d8beda1de6bebcc46e884d71754' > >>>> kernel_path = self.fetch_asset(kernel_url, > >>>> asset_hash=kernel_hash) > >>> > >> > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-03-03 18:54 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-02-28 22:43 [PATCH] tests/acceptance: Use imgtec.com URL for Fedora 22 artifacts Philippe Mathieu-Daudé 2021-03-01 13:30 ` Wainer dos Santos Moschetta 2021-03-01 13:56 ` Philippe Mathieu-Daudé 2021-03-02 15:27 ` Willian Rampazzo 2021-03-02 15:45 ` Philippe Mathieu-Daudé 2021-03-03 18:52 ` Willian Rampazzo
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.