ci: fix package installation in GitHub Actions

Message ID 20201219082610.31145-1-david.marchand@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series ci: fix package installation in GitHub Actions |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-testing success Testing PASS
ci/Intel-compilation success Compilation OK

Commit Message

David Marchand Dec. 19, 2020, 8:26 a.m. UTC
  APT cache must be updated to avoid trying to install an unavailable
version of a package.

Fixes: 87009585e293 ("ci: hook to GitHub Actions")

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
I did not find a way for the update to be done by GHA itself,
so adding an explicit step.

The robot hits this issue on all 32-bits builds at the moment.
I will apply this quickly.

---
 .github/workflows/build.yml | 2 ++
 1 file changed, 2 insertions(+)
  

Comments

David Marchand Dec. 21, 2020, 5:27 p.m. UTC | #1
On Sat, Dec 19, 2020 at 9:26 AM David Marchand
<david.marchand@redhat.com> wrote:
>
> APT cache must be updated to avoid trying to install an unavailable
> version of a package.
>
> Fixes: 87009585e293 ("ci: hook to GitHub Actions")
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Applied.
  
Andrew Boyer Dec. 30, 2020, 6 p.m. UTC | #2
Hello David,
I’m hitting this issue on the next/dpdk-next-net branch. Is there a way to pull the patch into all of the next-* branches so that others don’t hit it too?

For now I've pulled it into my own branch.

This Actions thing is cool. It’s a huge timesaver.

Thank you,
Andrew

> On Dec 21, 2020, at 12:27 PM, David Marchand <david.marchand@redhat.com> wrote:
> 
> On Sat, Dec 19, 2020 at 9:26 AM David Marchand
> <david.marchand@redhat.com> wrote:
>> 
>> APT cache must be updated to avoid trying to install an unavailable
>> version of a package.
>> 
>> Fixes: 87009585e293 ("ci: hook to GitHub Actions")
>> 
>> Signed-off-by: David Marchand <david.marchand@redhat.com>
> 
> Applied.
> 
> 
> -- 
> David Marchand
>
  
Ferruh Yigit Jan. 4, 2021, 12:26 p.m. UTC | #3
On 12/30/2020 6:00 PM, Andrew Boyer wrote:
> Hello David,
> I’m hitting this issue on the next/dpdk-next-net branch. Is there a way to pull the patch into all of the next-* branches so that others don’t hit it too?
> 

next-net rebased on the main repo to include the commit, fyi.

For the next trees, you can find their maintainers in the MAINTAINERS file.

> For now I've pulled it into my own branch.
> 
> This Actions thing is cool. It’s a huge timesaver.
> 
> Thank you,
> Andrew
> 
>> On Dec 21, 2020, at 12:27 PM, David Marchand <david.marchand@redhat.com> wrote:
>>
>> On Sat, Dec 19, 2020 at 9:26 AM David Marchand
>> <david.marchand@redhat.com> wrote:
>>>
>>> APT cache must be updated to avoid trying to install an unavailable
>>> version of a package.
>>>
>>> Fixes: 87009585e293 ("ci: hook to GitHub Actions")
>>>
>>> Signed-off-by: David Marchand <david.marchand@redhat.com>
>>
>> Applied.
>>
>>
>> -- 
>> David Marchand
>>
>
  
David Marchand Jan. 5, 2021, 8:49 a.m. UTC | #4
Hello Andrew,

On Wed, Dec 30, 2020 at 7:00 PM Andrew Boyer <aboyer@pensando.io> wrote:
>
> Hello David,
> I’m hitting this issue on the next/dpdk-next-net branch. Is there a way to pull the patch into all of the next-* branches so that others don’t hit it too?

Sorry, I was offline, next-* branches are maintained by others (as
explained by Ferruh).

>
> For now I've pulled it into my own branch.
>
> This Actions thing is cool. It’s a huge timesaver.

Happy to read that it helps.
  

Patch

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 05eb59527f..0b72df0ebe 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -87,6 +87,8 @@  jobs:
       with:
         path: reference
         key: ${{ steps.get_ref_keys.outputs.abi }}
+    - name: Update APT cache
+      run: sudo apt update
     - name: Install packages
       run: sudo apt install -y ccache libnuma-dev python3-setuptools
         python3-wheel python3-pip ninja-build libbsd-dev libpcap-dev