Build packages for client and yum repositories #64
Conversation
As discussed at Madrid workshop. Fixes quattor#16. Fixes quattor#17.
|
|
||
| [quattor-externals] | ||
| name=Quattor Externals | ||
| baseurl=http://yum.quattor.org/externals |
There was a problem hiding this comment.
this is not ok. we really need a repo with a $releasever , and maybe a second externals repo that is truly cross-OS like the quattor one.
same for aquilon repo imho (since it's python, i'm assuming the rpms have the python version in the packge path)?
There was a problem hiding this comment.
As we postponed solving these issues in 14.12, we should be able to rediscuss this now that we have a PR. I agree with @stdweird that we need to address the externals repo structure. I don't know what is referred to the "Quattor one" but if it is the one on quattorsrv.lal.in2p3.fr, I'd like to see it stopped in favor of one hosted at yum.quattor.org (see #62). This was only a temporary solution....
There was a problem hiding this comment.
@jouvin sorry again for being not clear enough. with the quattor one i meant the quattor repo specified above (ie the one with the released components)
There was a problem hiding this comment.
Something like:
/14.6.0/
/14.8.0/
/14.10.0/
/aquilon/el6/
/aquilon/el7/
/externals/el5/
/externals/el6/
/externals/el7/
Or the other way around?
/el5/externals/
/el6/aquilon
/el6/externals/
/el7/aquilon/
/el7/externals/
I wasn't aware of this one... |
|
http://quattorsrv.lal.in2p3.fr/packages/quattor/externals/... As said this was a temporary solution to be able to deliver obsolete RPMs still required by some components. I'll be happy to close it down after merging with an official one (see #62). |
Using naming consistent with template-library-standard.
|
Updated, please review 😅 |
| <parent> | ||
| <groupId>org.quattor.maven</groupId> | ||
| <artifactId>build-profile</artifactId> | ||
| <version>1.32</version> |
There was a problem hiding this comment.
Looks very old! Please update to last version...
|
With the exception of the comments above, LGTM. |
|
I believe this is ready for merging, I would like to test the modifications I need to make to the release scripts when I build RC2, so it would be good to have this in before then (tomorrow morning). |
| <defaultDirmode>755</defaultDirmode> | ||
| <mappings> | ||
| <mapping> | ||
| <directory>/etc/quattor/</directory> |
There was a problem hiding this comment.
See src/quattor-client/src/README
There was a problem hiding this comment.
i did, but if that is the only file, it should go in /usr/share/quattor, not /etc imho
afaik, /etc/quattor is not used by any quattor tools (but maybe it should 😄 )
There was a problem hiding this comment.
Maybe it should be /etc/quattor-release with a version number in it, I don't know.
There was a problem hiding this comment.
I agree. Looks as the most useful thing to do. Will be consistent with /etc/redhat-release. As soon as it is done, I'll be happy to merge it...
We should figure out a better way to fix this.
More useful than a stubby README.
|
@jouvin, done and tested! |
|
Will build RC4 (#96) when this is merged. |
Build packages for client and yum repositories
|
I hate having to merge my own stuff. |
|
Sorry missing it, I would happily have done it and in fact I was waiting for your green light to do it but missed your comment... |
As discussed at Madrid workshop.
Fixes #16.
Fixes #17.