Skip to content

Conversation

@zarybnicky
Copy link
Contributor

(created using eb --new-pr)

@verdurin
Copy link
Member

Test report by @verdurin
SUCCESS
Build succeeded for 4 out of 4 (1 easyconfigs in total)
nuc.lan - Linux Fedora 33, x86_64, Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz, Python 3.9.6
See https://gist.github.com/d900eac83c35ce1c6c1f380aba7da4dc for a full test report.

@verdurin
Copy link
Member

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@verdurin: Request for testing this PR well received on login1

PR test command 'EB_PR=13984 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_13984 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 6885

Test results coming soon (I hope)...

Details

- notification for comment with ID 918954795 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@verdurin verdurin added the new label Sep 14, 2021
@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cnx1 - Linux rocky linux 8.4, x86_64, Intel(R) Xeon(R) CPU E5-2690 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/e53ebf0f6a51a0bd1f0311b6c3746c8d for a full test report.

@verdurin
Copy link
Member

Test report by @verdurin
SUCCESS
Build succeeded for 5 out of 5 (1 easyconfigs in total)
easybuild-c7.novalocal - Linux centos linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake, IBRS), Python 3.6.8
See https://gist.github.com/8b5e0cd886a31f4dce67d8a4c82bda22 for a full test report.

@boegel boegel added this to the 4.x milestone Sep 14, 2021
Copy link
Member

@verdurin verdurin left a comment

Choose a reason for hiding this comment

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

Looks fine.

@verdurin verdurin modified the milestones: 4.x, next release (4.4.3?) Sep 14, 2021
@verdurin
Copy link
Member

Going in, thanks @zarybnicky!

@verdurin verdurin merged commit f0ddc13 into easybuilders:develop Sep 14, 2021
('bwapy', '0.1.4', {
'checksums': ['fcdad0c7311753d550b3a3aefbc2619380e7e35e7aeaa386fd491cba47e71824'],
}),
('nose', '1.3.7', {
Copy link
Member

Choose a reason for hiding this comment

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

@zarybnicky This is already included with Python itself, so can be removed here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll submit an update PR

('mappy', '2.17', {
'checksums': ['ed1460efc9c6785df28065b7e93e93c92227f623a181f1a852dca6e6acb1a15f'],
}),
('python-Levenshtein', '0.12.1', {
Copy link
Member

Choose a reason for hiding this comment

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

@zarybnicky We have dedicated easyconfigs for this, we should keep that up?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I looked for all deps other than python-Levenshtein :) I'll submit an update PR

@boegel
Copy link
Member

boegel commented Sep 14, 2021

follow-up PR to fix remarks: #13985

@zarybnicky zarybnicky deleted the 20210914094438_new_pr_CIRI-long102 branch September 14, 2021 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants