[PATCH v1 1/1] .github/workflows/codeql-analysis.yml: Add PIP caching

[PATCH v1 1/1] .github/workflows/codeql-analysis.yml: Add PIP caching

[Michael Kubacki]

From: Michael Kubacki <michael.kubacki@microsoft.com>

Adds caching of PIP dependencies. This reduces overall execution time
and decreases likelihood of a network error reaching out pypi to get
the dependencies.

Caching happens based on modules specified in pip-requirements.txt.

Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
---
 .github/workflows/codeql-analysis.yml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
index 791f79d1f014..33cc0bd6b46d 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -66,6 +66,8 @@ jobs:
       uses: actions/setup-python@v4
       with:
         python-version: '3.10.6'
+        cache: 'pip'
+        cache-dependency-path: 'pip-requirements.txt'
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-- 
2.40.0.windows.1

Re: [edk2-devel] [PATCH v1 1/1] .github/workflows/codeql-analysis.yml: Add PIP caching

[Michael D Kinney]

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Michael Kubacki
> Sent: Thursday, March 30, 2023 1:44 PM
> To: devel@edk2.groups.io
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Michael Kubacki <mikuback@linux.microsoft.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Subject: [edk2-devel] [PATCH v1 1/1] .github/workflows/codeql-analysis.yml: Add PIP caching
> 
> From: Michael Kubacki <michael.kubacki@microsoft.com>
> 
> Adds caching of PIP dependencies. This reduces overall execution time
> and decreases likelihood of a network error reaching out pypi to get
> the dependencies.
> 
> Caching happens based on modules specified in pip-requirements.txt.
> 
> Cc: Sean Brogan <sean.brogan@microsoft.com>
> Cc: Michael Kubacki <mikuback@linux.microsoft.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
> ---
>  .github/workflows/codeql-analysis.yml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
> index 791f79d1f014..33cc0bd6b46d 100644
> --- a/.github/workflows/codeql-analysis.yml
> +++ b/.github/workflows/codeql-analysis.yml
> @@ -66,6 +66,8 @@ jobs:
>        uses: actions/setup-python@v4
>        with:
>          python-version: '3.10.6'
> +        cache: 'pip'
> +        cache-dependency-path: 'pip-requirements.txt'
> 
>      # Initializes the CodeQL tools for scanning.
>      - name: Initialize CodeQL
> --
> 2.40.0.windows.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#102209): https://edk2.groups.io/g/devel/message/102209
> Mute This Topic: https://groups.io/mt/97960009/1643496
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [michael.d.kinney@intel.com]
> -=-=-=-=-=-=
> 

Re: [edk2-devel] [PATCH v1 1/1] .github/workflows/codeql-analysis.yml: Add PIP caching

[Michael D Kinney]

Merged:

PR: https://github.com/tianocore/edk2/pull/4227
Commit: https://github.com/tianocore/edk2/commit/b4af23aaab8a44341e43713a71cbebf23df2c27d

I verified that the caching is enabled and being used.  You can view the caches here:

https://github.com/tianocore/edk2/actions/caches

Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Thursday, March 30, 2023 2:08 PM
> To: devel@edk2.groups.io; mikuback@linux.microsoft.com
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: RE: [edk2-devel] [PATCH v1 1/1] .github/workflows/codeql-analysis.yml: Add PIP caching
> 
> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Michael Kubacki
> > Sent: Thursday, March 30, 2023 1:44 PM
> > To: devel@edk2.groups.io
> > Cc: Sean Brogan <sean.brogan@microsoft.com>; Michael Kubacki <mikuback@linux.microsoft.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > Subject: [edk2-devel] [PATCH v1 1/1] .github/workflows/codeql-analysis.yml: Add PIP caching
> >
> > From: Michael Kubacki <michael.kubacki@microsoft.com>
> >
> > Adds caching of PIP dependencies. This reduces overall execution time
> > and decreases likelihood of a network error reaching out pypi to get
> > the dependencies.
> >
> > Caching happens based on modules specified in pip-requirements.txt.
> >
> > Cc: Sean Brogan <sean.brogan@microsoft.com>
> > Cc: Michael Kubacki <mikuback@linux.microsoft.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
> > ---
> >  .github/workflows/codeql-analysis.yml | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
> > index 791f79d1f014..33cc0bd6b46d 100644
> > --- a/.github/workflows/codeql-analysis.yml
> > +++ b/.github/workflows/codeql-analysis.yml
> > @@ -66,6 +66,8 @@ jobs:
> >        uses: actions/setup-python@v4
> >        with:
> >          python-version: '3.10.6'
> > +        cache: 'pip'
> > +        cache-dependency-path: 'pip-requirements.txt'
> >
> >      # Initializes the CodeQL tools for scanning.
> >      - name: Initialize CodeQL
> > --
> > 2.40.0.windows.1
> >
> >
> >
> > -=-=-=-=-=-=
> > Groups.io Links: You receive all messages sent to this group.
> > View/Reply Online (#102209): https://edk2.groups.io/g/devel/message/102209
> > Mute This Topic: https://groups.io/mt/97960009/1643496
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub [michael.d.kinney@intel.com]
> > -=-=-=-=-=-=
> >