Skip to content

[skip ci] .github: apt-get -y update#630

Merged
marc-hb merged 2 commits intothesofproject:masterfrom
marc-hb:github-remove-apt-hack
Mar 15, 2021
Merged

[skip ci] .github: apt-get -y update#630
marc-hb merged 2 commits intothesofproject:masterfrom
marc-hb:github-remove-apt-hack

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Mar 13, 2021

Because Ubuntu VMs on github have a package index that is out of date by
design:

https://github.com/actions/virtual-environments/issues/ 1757

Signed-off-by: Marc Herbert marc.herbert@intel.com

@marc-hb marc-hb force-pushed the github-remove-apt-hack branch from 6834fe1 to 0b71c32 Compare March 13, 2021 18:26
@marc-hb marc-hb changed the title [skip ci] .github: remove workaround for out of date package index [skip ci] .github: apt-get -y update Mar 13, 2021
marc-hb added 2 commits March 13, 2021 10:36
Because it can save time.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Because Ubuntu VMs on github have a package index that is out of date by
design:

https://github.com/actions/virtual-environments/issues/ 1757

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb force-pushed the github-remove-apt-hack branch from 0b71c32 to 1a66225 Compare March 13, 2021 18:36
@marc-hb marc-hb marked this pull request as ready for review March 13, 2021 18:44
@marc-hb marc-hb requested a review from a team as a code owner March 13, 2021 18:44
run: sudo apt-get -y update

- name: get python libs
# FIXME: apt-get succeeds but 'import numpy' still fails!?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still have failure with "import numpy" after installing python3-numpy?
I installed with pip3 but it should be the same, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What happened here is pylint complaining that it cannot find python3-numpy. Maybe it was specific to pylint and maybe it was specific to github and maybe it's not happening anymore. Anyway it's off-topic in this PR :-)

steps:
- uses: actions/checkout@v2
- name: yamllint ourselves
run: yamllint .github/workflows/pull_request.yml
Copy link
Contributor

@fredoh9 fredoh9 Mar 15, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for being silly bit is this calling itself?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes this file checks itself. This will obviously not work if it's totally broken but even in that most extreme case it's still a good hint to run this locally.

Copy link
Contributor

@fredoh9 fredoh9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thank you for removing HACK

@marc-hb marc-hb merged commit 903892f into thesofproject:master Mar 15, 2021
@marc-hb marc-hb deleted the github-remove-apt-hack branch March 15, 2021 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants