search

apt-sim / AdePT

Accelerated demonstrator of electromagnetic Particle Transport
Apache License 2.0
25 stars 34 forks source link

cmake: remove duplicate MinSizeRel option #291

Closed andresailer closed 3 months ago

andresailer commented 3 months ago

I want to see if the Alma9 CI is working

phsft-bot commented 3 months ago

Can one of the admins verify this patch?

agheata commented 3 months ago

It did not trigger a build though. @andresailer do you want this merged now or do you still want to play with it?

andresailer commented 3 months ago

I want to see the build run, I don't want this merged before the build ran successfully on alma9.

(I just ran a spellcheck on CMakeLists.txt, but I would have also just done a dummy change)

agheata commented 3 months ago

I want to see the build run, I don't want this merged before the build ran successfully on alma9.

OK, no problem, let me know if anything you may need

andresailer commented 3 months ago

I think someone has to tell the bot to run the pipeline?

andresailer commented 3 months ago

@adept-bot bot build

andresailer commented 3 months ago

@phsft-bot bot build

andresailer commented 3 months ago

sft-bot build

andresailer commented 3 months ago

@phsft-bot build

andresailer commented 3 months ago

@agheata Can you tell the bot to build?

agheata commented 3 months ago

I don't see the pipeline in the AdePT-CI. It normally starts automatically, maybe it detected that no dependencies were changed so no build is needed (just shooting in the dark)

andresailer commented 3 months ago

I think I am not in some list of developers, so the bot doesn't start the pipeline unless someone verifies somehow

agheata commented 3 months ago

In such case it should prompt me to start the pipeline, which it doesn't. I managed to restart the pipeline for #289 https://lcgapp-services.cern.ch/spi-jenkins/job/AdePT-CI/661/

agheata commented 3 months ago

Which runs happily on centos7 Running on lcgapp-centos7-x86-64-gpu-03 in /build/sftnight/workspace/AdePT-CI

agheata commented 3 months ago

I just pushed to your branch, let's see

agheata commented 3 months ago

I just pushed to your branch, let's see

OK, so even if I push, it won't start the pipeline

andresailer commented 3 months ago

Can you say @phsft-bot build Without the tick marks?

agheata commented 3 months ago

@phsft-bot build

agheata commented 3 months ago

@phsft-bot rebuild

agheata commented 3 months ago

Nothing... The project is configured to send notifications to the webhook: https://github.com/apt-sim/AdePT/settings/hooks/277932760

andresailer commented 3 months ago

Thanks for trying!

andresailer commented 3 months ago

@phsft-bot test this please

andresailer commented 3 months ago

@phsft-bot retest this please

agheata commented 3 months ago

@andresailer I see that the build was done, but still on centos7 ? Running on lcgapp-centos7-x86-64-gpu-02 in /build/sftnight/workspace/AdePT-CI

andresailer commented 3 months ago

No, this is alma9, the name of the machine is just the same as before.

agheata commented 3 months ago

OK, then it should be working.

andresailer commented 3 months ago

Then I will also migrate the other GPU machine to alma9?

agheata commented 3 months ago

Then I will also migrate the other GPU machine to alma9?

Yes, why not

agheata commented 3 months ago

@andresailer it seems that now all nodes labeled cuda10 have disappeared, so the VecGeom pipeline stalls

agheata commented 3 months ago

@andresailer I guess we can close this since the CI is migrated?

andresailer commented 3 months ago

Yes, you can also merge it :)