Skip to content
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

Unbreak GHDL nightly CI #2947

Merged
merged 1 commit into from
Apr 24, 2022
Merged

Conversation

imphil
Copy link
Member

@imphil imphil commented Apr 24, 2022

GHDL nightly builds for Ubuntu 20.04 are not provided for mcode any
more, but only for llvm. Let's switch to that.

GHDL nightly builds for Ubuntu 20.04 are not provided for mcode any
more, but only for llvm. Let's switch to that.
@imphil
Copy link
Member Author

imphil commented Apr 24, 2022

GHDL nightly is fixed by this PR, but we see another error with Python 3.11. I'm not yet sure where the failure really comes from, but we at least have to look into

2022-04-24T11:42:40.7823855Z Exception ignored in: <_io.FileIO name=2 mode='wb' closefd=True>
2022-04-24T11:42:40.7824437Z ResourceWarning: unclosed file <_io.TextIOWrapper name=2 mode='w' encoding='UTF-8'>

@codecov
Copy link

codecov bot commented Apr 24, 2022

Codecov Report

Merging #2947 (bd88815) into master (41f2ab7) will increase coverage by 0.44%.
The diff coverage is 100.00%.

❗ Current head bd88815 differs from pull request most recent head 3e40f0f. Consider uploading reports for the commit 3e40f0f to get more accurate results

@@            Coverage Diff             @@
##           master    #2947      +/-   ##
==========================================
+ Coverage   59.90%   60.34%   +0.44%     
==========================================
  Files          28       28              
  Lines        5848     5848              
  Branches     1521     1392     -129     
==========================================
+ Hits         3503     3529      +26     
+ Misses       2064     2035      -29     
- Partials      281      284       +3     
Impacted Files Coverage Δ
cocotb/__init__.py 70.12% <100.00%> (-0.44%) ⬇️
cocotb/binary.py 36.02% <0.00%> (+0.93%) ⬆️
cocotb/runner.py 54.20% <0.00%> (+1.73%) ⬆️
cocotb/handle.py 66.38% <0.00%> (+1.95%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41f2ab7...3e40f0f. Read the comment docs.

@ktbarrett
Copy link
Member

ktbarrett commented Apr 24, 2022

3.11 failure actually due to an issue with coverage package https://github.com/cocotb/cocotb/runs/6147105555?check_suite_focus=true#step:25:976.

@imphil
Copy link
Member Author

imphil commented Apr 24, 2022

3.11 failure actually due to an issue with coverage package cocotb/cocotb/runs/6147105555?check_suite_focus=true#step:25:976.

Ah, good catch. I missed that one when scrolling through. I filed #2949, as it's unrelated to what this PR is trying to accomplish.

@imphil
Copy link
Member Author

imphil commented Apr 24, 2022

And I filed #2950 for the ResourceWarning.

@imphil imphil requested a review from a team April 24, 2022 15:32
@ktbarrett
Copy link
Member

Is there an issue in GHDL or some announcement from them as to why they aren't providing the mcode backend in the nightly builds anymore?

@imphil
Copy link
Member Author

imphil commented Apr 24, 2022

Is there an issue in GHDL or some announcement from them as to why they aren't providing the mcode backend in the nightly builds anymore?

I haven't seen anything. But I'm slightly annoyed by the repeated breakage of the GHDL nightly regression anyways, maybe it's time to remove it -- there's little point in testing unstable code in a PR against unstable simulators.

@imphil imphil merged commit fb6e5cb into cocotb:master Apr 24, 2022
@imphil imphil deleted the ghdl-ci-fix-nightly-llvm branch April 24, 2022 20:22
imphil added a commit to imphil/cocotb that referenced this pull request May 17, 2022
GHDL Nightly build artifacts are not reliably available and have caused
frequent build failures. Furthermore, testing an unstable (to-be-tested)
version of cocotb against a unknown version of a simulator isn't a good
match for a CI environment, where we want to see if the changeset broke
something, not if a simulator update broke something.

See also
* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit to imphil/cocotb that referenced this pull request May 17, 2022
GHDL Nightly build artifacts are not reliably available and have caused
frequent build failures. Furthermore, testing an unstable (to-be-tested)
version of cocotb against a unknown version of a simulator isn't a good
match for a CI environment, where we want to see if the changeset broke
something, not if a simulator update broke something.

See also
* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit to imphil/cocotb that referenced this pull request May 18, 2022
GHDL Nightly build artifacts are not reliably available and have caused
frequent build failures. Furthermore, testing an unstable (to-be-tested)
version of cocotb against a unknown version of a simulator isn't a good
match for a CI environment, where we want to see if the changeset broke
something, not if a simulator update broke something.

See also
* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit to imphil/cocotb that referenced this pull request May 18, 2022
GHDL Nightly build artifacts are not reliably available and have caused
frequent build failures. Furthermore, testing an unstable (to-be-tested)
version of cocotb against a unknown version of a simulator isn't a good
match for a CI environment, where we want to see if the changeset broke
something, not if a simulator update broke something.

See also
* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit to imphil/cocotb that referenced this pull request May 18, 2022
GHDL Nightly build artifacts are not reliably available and have caused
frequent build failures. Furthermore, testing an unstable (to-be-tested)
version of cocotb against a unknown version of a simulator isn't a good
match for a CI environment, where we want to see if the changeset broke
something, not if a simulator update broke something.

See also
* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit to imphil/cocotb that referenced this pull request May 18, 2022
Split the GHDL testing into two buckets:
- Use a stable release version of GHDL for CI (pull request) testing.
- Test the latest master of GHDL in the extended tests.

GHDL 2.0 is the first version that works with cocotb. Use this version
for CI testing, and document this fact.

For experimental testing, use the GHDL master from Git and compile it
ourselves. The GHDL nightly build artifacts (and the associated GitHub
actions) have been unreliable for us in the past, and provide no real
value. See also

* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit to imphil/cocotb that referenced this pull request May 18, 2022
Split the GHDL testing into two buckets:
- Use a stable release version of GHDL for CI (pull request) testing.
- Test the latest master of GHDL in the extended tests.

GHDL 2.0 is the first version that works with cocotb. Use this version
for CI testing, and document this fact.

For experimental testing, use the GHDL master from Git and compile it
ourselves. The GHDL nightly build artifacts (and the associated GitHub
actions) have been unreliable for us in the past, and provide no real
value. See also

* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit to imphil/cocotb that referenced this pull request May 22, 2022
Split the GHDL testing into two buckets:
- Use a stable release version of GHDL for CI (pull request) testing.
- Test the latest master of GHDL in the extended tests.

GHDL 2.0 is the first released version that passes our test suite (as of
today). It also happens to be the latest released version. Use this
version for CI testing, and document the fact that earlier versions
don't pass our test suite (earlier version might still work for certain
scenarios).

For experimental testing, use the GHDL master from Git and compile it
ourselves. The GHDL nightly build artifacts (and the associated GitHub
actions) have been unreliable for us in the past, and provide no real
value. See also

* cocotb#2947
* cocotb#2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
imphil added a commit that referenced this pull request May 23, 2022
Split the GHDL testing into two buckets:
- Use a stable release version of GHDL for CI (pull request) testing.
- Test the latest master of GHDL in the extended tests.

GHDL 2.0 is the first released version that passes our test suite (as of
today). It also happens to be the latest released version. Use this
version for CI testing, and document the fact that earlier versions
don't pass our test suite (earlier version might still work for certain
scenarios).

For experimental testing, use the GHDL master from Git and compile it
ourselves. The GHDL nightly build artifacts (and the associated GitHub
actions) have been unreliable for us in the past, and provide no real
value. See also

* #2947
* #2926
* Right now, GHDL nightly is failing because the GHDL CI failed to
  upload an Ubuntu 20.04 binary
  (https://github.com/ghdl/ghdl/runs/6475779323?check_suite_focus=true).
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.

None yet

2 participants