Skip to content

GitHub Action: Bump actions/setup-python from 4 to 5 #190

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 359 commits into
base: dasharo
Choose a base branch
from

Conversation

dependabot[bot]
Copy link

@dependabot dependabot bot commented on behalf of github Nov 25, 2024

Bumps actions/setup-python from 4 to 5.

Release notes

Sourced from actions/setup-python's releases.

v5.0.0

What's Changed

In scope of this release, we update node version runtime from node16 to node20 (actions/setup-python#772). Besides, we update dependencies to the latest versions.

Full Changelog: actions/setup-python@v4.8.0...v5.0.0

v4.8.0

What's Changed

In scope of this release we added support for GraalPy (actions/setup-python#694). You can use this snippet to set up GraalPy:

steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v4 
  with:
    python-version: 'graalpy-22.3' 
- run: python my_script.py

Besides, the release contains such changes as:

New Contributors

Full Changelog: actions/setup-python@v4...v4.8.0

v4.7.1

What's Changed

Full Changelog: actions/setup-python@v4...v4.7.1

v4.7.0

In scope of this release, the support for reading python version from pyproject.toml was added (actions/setup-python#669).

      - name: Setup Python
        uses: actions/setup-python@v4
</tr></table> 

... (truncated)

Commits
  • 0b93645 Enhance workflows: Add macOS 13 support, upgrade publish-action, and update d...
  • 9c76e71 Bump pillow from 7.2 to 10.2.0 in /tests/data (#956)
  • f4c5a11 Revise isGhes logic (#963)
  • 19dfb7b Bump default versions to latest (#905)
  • e9675cc Merge pull request #943 from actions/Jcambass-patch-1
  • 3226af6 Upgrade IA publish
  • 70dcb22 Merge pull request #941 from actions/Jcambass-patch-1
  • 65b48c7 Create publish-immutable-actions.yml
  • 29a37be initial commit (#938)
  • f677139 Bump pyinstaller from 3.6 to 5.13.1 in /tests/data (#923)
  • Additional commits viewable in compare view

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

Stojek139808 and others added 30 commits June 15, 2024 15:31
…tures.c: Remove extra newline

Co-authored-by: Michał Żygowski <[email protected]>
…ge for handoff at ReadyToBoot

Signed-off-by: Kacper Stojek <[email protected]>
…ar to a single uint8 value

Signed-off-by: Kacper Stojek <[email protected]>
…t of PcdVTdPolicyPropertyMask

Signed-off-by: Kacper Stojek <[email protected]>
…wrong values in IommuConfig

Signed-off-by: Kacper Stojek <[email protected]>
…tures.c: disable IOMMU by default

Signed-off-by: Michał Żygowski <[email protected]>
…ot enabled

Signed-off-by: Michał Żygowski <[email protected]>
Co-authored-by: Michał Kopeć <[email protected]>
…turesVfr.vfr: fix default and add reset requirement

Signed-off-by: Michał Żygowski <[email protected]>
…tures.c: handle interactive NetworkBoot default

Signed-off-by: Michał Żygowski <[email protected]>
… WDT timeout depend on PCD

Signed-off-by: Michał Żygowski <[email protected]>
…tures: add FUM

This commit adds option in security menu to enable Firmware Update Mode.
As a result, UEFI variable is set, which will be later caught by
coreboot and acted upon to disable firmware write protections.

Signed-off-by: Krystian Hebel <[email protected]>
Signed-off-by: Michał Żygowski <[email protected]>
Selects state the system gets into when power supply is restored after
power failure.

Adds gDasharoSystemFeaturesTokenSpaceGuid.PcdDefaultPowerFailureState
PCD which sets its default value.

Signed-off-by: Sergii Dmytruk <[email protected]>
It controls visibility of sleep type inside Power Management Options
submenu.

The PCD is on by default for compatibility.

Signed-off-by: Sergii Dmytruk <[email protected]>
…tures.c: warn about network use

DTS will be automatically started after FUM is enabled. Inform user that
Ethernet cable must be plugged in for seamless update process.

Signed-off-by: Krystian Hebel <[email protected]>
…Fi + BT radios setting

Signed-off-by: Mixss <[email protected]>
Signed-off-by: Michał Kopeć <[email protected]>
philipanda and others added 26 commits July 26, 2024 10:25
This gives a caller a chance to report a meaningful error to the user.

Signed-off-by: Sergii Dmytruk <[email protected]>
Commit b68d566 added support for
input subject name with signtool and broke --decode operation by using
incorrect identifier in one place (could be an incomplete rename during
review).

It's `args.SignToolSubjectName`, not `args.SignSubjectName`.

Signed-off-by: Sergii Dmytruk <[email protected]>
This field seems to be one-based according UEFI specification, default
value is 1 and --decode of GenerateCapsule.py errors upon seeing
UpdateImageIndex less than 1.  So align --encode behaviour to enforce a
value within the 1..255 range.

Signed-off-by: Sergii Dmytruk <[email protected]>
Print error text from the exception.

Signed-off-by: Sergii Dmytruk <[email protected]>
--decode unconditionally uses args.OutputFile.name as a prefix for
output files that it creates and fails in a non-pretty way without
--output option.

This doesn't address creation/truncation of the file specified via
--output, but at least you're able to decode a capsule.

Signed-off-by: Sergii Dmytruk <[email protected]>
Just add a space between colon and a more detailed error message in two
places.

Signed-off-by: Sergii Dmytruk <[email protected]>
struct.unpack() returns a tuple even for a single-element pack,
resulting in signature verification being evaluated to false even when
the signature is there.

This fixes --decode and --dump-info actions incorrectly reporting issues
with parsing capsule dependencies when there are none.

Signed-off-by: Sergii Dmytruk <[email protected]>
…he INF file

Fix build complaining on missing ParseVBootWorkbuf.

Signed-off-by: Michał Żygowski <[email protected]>
Add PlatformGopPolicy to use external GOP driver

Signed-off-by: Matt DeVillier <[email protected]>
Use PlatformGopPolicy in place of GraphicsOutputDxe if command
line param passed

Signed-off-by: Matt DeVillier <[email protected]>
Signed-off-by: Michał Żygowski <[email protected]>
Signed-off-by: Michał Żygowski <[email protected]>
Make gmtime() return pointer to a statically allocated storage.  This
is how it's documented and that's how it gets used.  The API isn't
thread-safe but given that EDK2 is essentially single-threaded (MP
Services Protocol is a special exception).

Given that this code gets run at runtime phase, the leak could
potentially be used to exhaust memory reserved for the runtime phase.

Signed-off-by: Sergii Dmytruk <[email protected]>
This is a follow up for 6ca2060
and 784750e which provide more detailed
information on the issue and how this addresses it.

The files modified by this commit were chosen based on the list of
sources in CryptoPkg/Library/BaseCryptLibMbedTls/RuntimeCryptLib.inf

The only source permitted to request memory from boot services is
SysCall/RuntimeMemAllocation.c which does it in constructor before
ExitBootServices() is called.

Trying to update minimal set of files because some of the API which does
allocations get used outside of BaseCryptLibMbedTls and can do
FreePool().  In the updated files, allocations are of two types:
 - temporary allocations within a function (they don't get returned or
   set to some output parameter)
 - paired alloc/free kind of functions which remain in control of how
   the memory is treated

Signed-off-by: Sergii Dmytruk <[email protected]>
This is a port of upstream commits c248802 and bfefdc2

On modern platforms with TBT devices the coreboot resource allocator
opens large PCI bridge MMIO windows above 4GiB to place hotplugable
PCI BARs there as they won't fit below 4GiB. In addition modern
GPGPU devices have very big PCI bars that doesn't fit below 4GiB.

The PciHostBridgeLib made lots of assumptions about the coreboot
resource allocator that were not verified at runtime and are no
longer true.

Remove all of the 'coreboot specific' code and implement the same
logic as OvmfPkg's ScanForRootBridges.

Fixes assertion
"ASSERT [PciHostBridgeDxe] Bridge->Mem.Limit < 0x0000000100000000ULL".

Co-authored-by: Patrick Rudolph <[email protected]>
Signed-off-by: Michał Kopeć <[email protected]>
Tcg2PhysicalPresenceLibSubmitRequestToPreOSFunction writes to TPM2
physical presence PPI provided by coreboot (a memory region preserved
across reboots). CPU caches must be explicitly flushed prior to platform
reboot or request written to PPI will be lost.

Signed-off-by: Artur Kowalski <[email protected]>
Flush cache not only when placing request in PPI, but also after
clearing old request from PPI.

Signed-off-by: Artur Kowalski <[email protected]>
It has been observed on MinnowBoard Turbot that the detected CPU
count is lower than the number of all cores. In the tested unit, CPU is
dual core, so only the BSP is detected. However, similar situation is
observed on MTL laptop, where a total of 22 cores should be reported,
but only 1 is detected.

After EDK2 rebase a new PCD has been added, PcdFirstTimeWakeUpAPsBySipi
which, when enabled (by default), sends only SIPI to APs. When disabled
sends full INIT-SIPI-SIPI sequence. Only the latter case causes all the
APs to wake up and be detected.

Signed-off-by: Michał Żygowski <[email protected]>
Signed-off-by: Sergii Dmytruk <[email protected]>
This fixes "SecurityPkg: measure Dasharo variables before boot".

gRT->GetNextVariableName() doesn't return variables in any fixed order.
Seems like the order matches order in SMMSTORE.  This means that
measuring variables while enumerating them will produce different
results depending on which variables were update last (setting a
variable in SMMSTORE is marking old entry as deleted and appending of a
new one).  Sort list of variables that share the same GUID before
measuring any of them to impose a fixed order.

Also fix spacing in several places.

Signed-off-by: Sergii Dmytruk <[email protected]>
Match the help text string with what can actually be set at runtime.

Signed-off-by: Michał Żygowski <[email protected]>
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Nov 25, 2024
Copy link
Author

dependabot bot commented on behalf of github Nov 25, 2024

Dependabot tried to add @makubacki, @mdkinney and @spbrogan as reviewers to this PR, but received the following error from GitHub:

POST https://api.github.com/repos/Dasharo/edk2/pulls/190/requested_reviewers: 422 - Reviews may only be requested from collaborators. One or more of the users or teams you specified is not a collaborator of the Dasharo/edk2 repository. // See: https://docs.github.com/rest/pulls/review-requests#request-reviewers-for-a-pull-request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.