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

jobs-builder: plan of migration - your help is needed #407

Open
wainersm opened this issue Nov 10, 2021 · 3 comments
Open

jobs-builder: plan of migration - your help is needed #407

wainersm opened this issue Nov 10, 2021 · 3 comments
Labels
feature New functionality needs-review Needs to be assessed by the team.

Comments

@wainersm
Copy link
Contributor

Context

Recently it was merged #359 (main - #406) which introduced the basics for allowing us to manage the creation and update of Jenkins jobs in http://jenkins.katacontainers.io with the Jenkins Job Builder (JJB) tool. See more information on README.md.

The main benefits of using JJB on our project are:
a) avoid (hard to) reviews on Jenkins XMLs (which are machine-readable jobs representations, not meant for humans)
a) changes will be reviewed and approved before they are commited on Jenkins (just like any other piece of code in the project);
b) changes will be more easily traceable;
c) multiple similar jobs can be generated out of templates

In order to have jobs managed by JJB, their definitions should be on one of the yaml files in the jobs-builder/jobs directory. So far there are only 4 jobs managed with JJB, those are the cache jobs that I maintain; but, IMHO, ideally we should be using that tool to manage all the jobs. Therefore the goal of this issue is to come up with a migration plan.

Pre-req

First of all, I'd like to know whether we have consensus about the use of JJB or not. IMHO all the job owners on the always green CI initiative should agree with that.

Also if anyone strongly opposes to this initiative and/or has a different opinion, that's the proper time to let us know.

Plan

(Draft)

  • Announce on kata-dev mailing list and developers meetings
  • Define what jobs will be managed with JJB
  • Define milestones and timeline

Additional resources

/cc @kata-containers/architecture-committee @kata-containers/ci @ariel-adam @chavafg @kata-containers/redhat @jodh-intel @Jakob-Naucke @jongwu @ananos

@wainersm wainersm added feature New functionality needs-review Needs to be assessed by the team. labels Nov 10, 2021
@Jakob-Naucke
Copy link
Member

I'm in favour of using JJB, I think we can reduce LOC tremendously and make all changes traceable.

@jodh-intel
Copy link
Contributor

+1 from me. Anything to avoid that hellish XML config! ;)

@fidencio
Copy link
Member

Huge +1 from me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality needs-review Needs to be assessed by the team.
Projects
None yet
Development

No branches or pull requests

4 participants