ibzib commented on PR #17239:
URL: https://github.com/apache/beam/pull/17239#issuecomment-1090864191

   > Perhaps the seed job just doesn't create itself any more? That would 
probably be smart.
   
   Whoa, judging from the logs the seed job does generate itself. That 
definitely doesn't seem ideal. 
   
   >     GeneratedJob{name='beam_SeedJob'}
   >     GeneratedJob{name='beam_SeedJob_Standalone'}
   
   > Ah, and specifically, the seed job shared "library" code with all the 
other jobs, so it could be broken by breakage in the shared groovy files.
   
   At least it looks like it doesn't share code with other jobs anymore. The 
only thing it imports is Committers.groovy, which is just a placeholder.
   
   
https://github.com/apache/beam/blob/f90379a67f9cbc89425cd161452cdf6172354e93/.test-infra/jenkins/job_00_seed.groovy#L21
   
   > I expect it is out of date as well.
   
   It's only trivially different. 
   
   ```diff
   $ diff .test-infra/jenkins/job_seed_standalone.groovy 
.test-infra/jenkins/job_00_seed.groovy
   
   23c23
   < job('beam_SeedJob_Standalone') {
   —
   > job('beam_SeedJob') {
   77c77
   < cron('0 */5 * * *')
   —
   > cron('H */6 * * *')
   85c85
   < triggerPhrase('Run Standalone Seed Job')
   —
   > triggerPhrase('Run Seed Job')
   90c90
   < context("Jenkins: Standalone Seed Job")
   —
   > context("Jenkins: Seed Job")
   110c110
   < python3.8 -m venv ve3 && source ./ve3/bin/activate &&
   —
   > python3.8 -m venv ve3 && source ve3/bin/activate &&
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to