Skip to content

Comments

switch bamtofastq to Cargo easyblock#17595

Merged
boegel merged 3 commits intoeasybuilders:developfrom
Micket:20230322180124_new_pr_bamtofastq140
May 24, 2023
Merged

switch bamtofastq to Cargo easyblock#17595
boegel merged 3 commits intoeasybuilders:developfrom
Micket:20230322180124_new_pr_bamtofastq140

Conversation

@Micket
Copy link
Contributor

@Micket Micket commented Mar 22, 2023

(created using eb --new-pr)

Requires:

but apparently, it needs more work first, it's not finding the vendored sources for some reason

@Micket Micket added this to the next release (4.7.2?) milestone Mar 22, 2023
@Micket
Copy link
Contributor Author

Micket commented Mar 22, 2023

@boegel looks like this needs more work, it's not finding vendored sources for some reason

@Micket
Copy link
Contributor Author

Micket commented Mar 22, 2023

So, i cheated a bit, renamed the github source tarball so that my checksum json thing in cargo easyblock will work. Not very robust strategy though.

@boegel
Copy link
Member

boegel commented Mar 27, 2023

@Micket Let's discuss this in the EasyBuild conf call this week?

@Micket
Copy link
Contributor Author

Micket commented Apr 8, 2023

@boegelbot please test @ generoso
EB_ARGS="--include-easyblocks-from-pr 2902"

@Micket
Copy link
Contributor Author

Micket commented Apr 8, 2023

Test report by @Micket
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2902
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
vera-c1 - Linux Rocky Linux 8.6, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/Micket/b728ecd040d396c3e6e46647493539f3 for a full test report.

@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=17595 EB_ARGS="--include-easyblocks-from-pr 2902" EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_17595 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1500912080 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).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2902
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/c55ee2642fc8ec909ff82aeaabd44012 for a full test report.

@boegel boegel changed the title Switch bamtofastq to cargo easyblock switch bamtofastq to Cargo easyblock Apr 12, 2023
@Micket
Copy link
Contributor Author

Micket commented Apr 13, 2023

Test report by @Micket
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2902
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
vera-c1 - Linux Rocky Linux 8.6, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/Micket/30e7f5c42de5aa9cb0fb7b01f863c95f for a full test report.

@Micket
Copy link
Contributor Author

Micket commented May 10, 2023

Test report by @Micket
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2902
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
vera-c1 - Linux Rocky Linux 8.7, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/Micket/c610e966822dc14c8a29486c4911f048 for a full test report.

@boegel
Copy link
Member

boegel commented May 23, 2023

@boegelbot please test @ jsc-zen2
EB_ARGS="--include-easyblocks-from-pr 2902"

@boegelbot
Copy link
Collaborator

@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=17595 EB_ARGS="--include-easyblocks-from-pr 2902" /opt/software/slurm/bin/sbatch --mem-per-cpu=4000M --job-name test_PR_17595 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1559770393 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).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2902
FAILED
Build succeeded for 1 out of 2 (1 easyconfigs in total)
jsczen2c1.int.jsc-zen2.easybuild-test.cluster - Linux Rocky Linux 8.5, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/boegelbot/265e84b1fd062a3a0baa106489f1e067 for a full test report.

@boegel
Copy link
Member

boegel commented May 23, 2023

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2902
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3165.skitty.os - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/boegel/6e80af569ada1983b47564f8f793121e for a full test report.

@boegel
Copy link
Member

boegel commented May 24, 2023

@boegelbot please test @ generoso
EB_ARGS="--include-easyblocks-from-pr 2902"

@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=17595 EB_ARGS="--include-easyblocks-from-pr 2902" EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_17595 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1560485095 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).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2902
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns2 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/468cdacc550cba8e0b4d595065fc66da for a full test report.

@boegel
Copy link
Member

boegel commented May 24, 2023

@boegelbot please test @ generoso
EB_ARGS="--include-easyblocks-from-pr 2934"

@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=17595 EB_ARGS="--include-easyblocks-from-pr 2934" EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_17595 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1561321067 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).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2934
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns5 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/85fa701e35e949dc829c71b9614e75de for a full test report.

@boegel boegel force-pushed the 20230322180124_new_pr_bamtofastq140 branch from af6542c to 10412a6 Compare May 24, 2023 15:58
@easybuilders easybuilders deleted a comment from boegelbot May 24, 2023
Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Member

boegel commented May 24, 2023

Going in, thanks @Micket!

@boegel boegel merged commit 841d3d5 into easybuilders:develop May 24, 2023
@Micket Micket deleted the 20230322180124_new_pr_bamtofastq140 branch October 11, 2023 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants