smoser | blackboxsw, http://paste.ubuntu.com/24804339/ | 00:04 |
---|---|---|
=== rangerpb is now known as rangerpbzzzz | ||
smoser | rharper, where does the setuptools depends come from ? | 04:36 |
rharper | no where, we need to explicitly depend on it | 04:36 |
rharper | in el6, it gets pulled in via some other package dep but we BuildRequires we should also just Requires it as well | 04:37 |
rharper | we had it fixed earlier today | 04:37 |
smoser | i dont know.. i dont think i ever had that in there. | 04:38 |
rharper | hrm | 04:38 |
smoser | i think that requirements.txt file is getting written with it in there. | 04:38 |
smoser | while we do not need it | 04:38 |
rharper | test-requirements.txt:setuptools | 04:39 |
rharper | 04:39 | |
rharper | not sure if that's getting picked up; in any case, we *do* need it since the loader requires it (/usr/bin/cloud-init calls it for pkg_resource) | 04:39 |
smoser | hm.. | 04:40 |
smoser | test-requirements.txt says | 04:40 |
smoser | # Only really needed on older versions of python | 04:40 |
smoser | contextlib2 | 04:40 |
smoser | setuptools | 04:40 |
rharper | I'm just grepping around, I don't know how we got setuptools into the previous build that worked, but it's not installed now via yum install cloud-init | 04:42 |
rharper | I thought we agreed it's a requirement as long as we're using setup.py | 04:42 |
smoser | no. i think that was pkg_resources | 04:43 |
smoser | i forget. | 04:43 |
smoser | what a pain | 04:43 |
rharper | pkg_resources comes from setuptools | 04:44 |
smoser | ok. i added that to spec | 04:56 |
smoser | and pushed | 04:56 |
smoser | rharper, ^ | 05:04 |
rharper | k | 05:04 |
smoser | and wih that... /me goes to bed. | 05:04 |
smoser | ci is happy with it now too. | 05:04 |
smoser | good night | 05:04 |
rharper | k | 05:04 |
rharper | https://copr.fedorainfracloud.org/coprs/g/cloud-init/cloud-init/build/562783/ | 05:12 |
rharper | smoser: that's working on el6 and el7 =) | 05:12 |
=== frickler_ is now known as frickler | ||
=== sambetts|afk is now known as sambetts | ||
smoser | rharper, i split into 2 merge proposals | 14:36 |
smoser | https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325311 | 14:36 |
smoser | https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325192 | 14:36 |
rharper | ok | 14:36 |
smoser | ie, cloud-config in one and redhat spec in another. | 14:36 |
rharper | ack | 14:36 |
smoser | one question... i dont know if we need the centos variant. | 14:36 |
smoser | do we? | 14:36 |
* smoser walks down | 14:36 | |
rharper | yes, for the cloud-config | 14:36 |
rharper | we use the distro.variant to set the distro value as well as the default user name (and Gecos values) | 14:37 |
larsks | smoser: dumb question: should I expect cloud-init to handle network_data.json when booting with a network data source, rather than config drive? | 16:30 |
smoser | larsks, not yet | 16:36 |
smoser | really, really want to do that... | 16:36 |
smoser | but not at the moment :-( | 16:36 |
larsks | smoser: thanks, just wanted to make sure I wasn't crazy. | 16:36 |
smoser | larsks, the goal is to make the openstack datasource work like the digital ocean one does. | 16:38 |
smoser | we'd identify (via dmi data) that we are running on penstack. | 16:38 |
smoser | and then raise an interface with the link local address, and hit the metadata service | 16:38 |
smoser | get the network data | 16:38 |
smoser | take the nic down | 16:38 |
smoser | apply the network_data.json | 16:38 |
larsks | That makes sense. | 16:39 |
smoser | rharper, | 16:49 |
smoser | https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325192 | 16:49 |
smoser | addressed all your things there | 16:49 |
=== sambetts is now known as sambetts|afk | ||
rharper | smoser: ok, trunk needs this http://paste.ubuntu.com/24809440/ | 18:00 |
rharper | smoser: http://paste.ubuntu.com/24809461/ look ok ? | 18:02 |
smoser | powersj, http://paste.ubuntu.com/24809515/ | 18:13 |
smoser | powersj, http://paste.ubuntu.com/24809682/ | 18:48 |
powersj | rharper: https://paste.ubuntu.com/24810376/ | 20:43 |
rharper | powersj: nice! | 20:46 |
=== cpaelzer_ is now known as cpaelzer | ||
smoser | larsks, we're trying to get spec file going in trunk. it builds happily now in copr and we get rpms out. | 23:03 |
smoser | horay! | 23:03 |
smoser | but we would also like the things to actually *work*. :) | 23:04 |
smoser | and that is causing a problem in that right now we're not getting systemd stuff enabled in the spec file. | 23:04 |
larsks | smoser: "not getting enabled" == "you are running systemctl enable but service isn't starting"? Or something else? | 23:05 |
rharper | %define use_systemd (0%{?fedora} && 0%{?fedora} >= 18) || (0%{?rhel} && 0%{?rhel} >= 7) || (0%{?suse_version} && 0%{?suse_version} >=1210) | 23:05 |
rharper | so, in specfile, that sets a flag to use systemd based on the dist version | 23:06 |
larsks | Sure. | 23:06 |
rharper | I think we need that | 23:06 |
rharper | larsks: does that look sane to add ? | 23:06 |
rharper | then we do an if %use_systemd and BuildRequires systemd | 23:07 |
smoser | http://paste.ubuntu.com/24811338/ | 23:07 |
larsks | rharper: It seems like a reasonable idea. Is it not working? | 23:07 |
smoser | spec ends up looking like ^ | 23:07 |
* larsks looks | 23:07 | |
rharper | larsks: using what smoser posted | 23:07 |
rharper | but I thought that rhel7 builder chroots would have systemd in it by default | 23:07 |
rharper | if we switch to my post, then we need to buildrequires systemd | 23:07 |
rharper | based on version, I think we'll need to do that | 23:07 |
powersj | COPR builds pushed by jenkins: https://jenkins.ubuntu.com/server/job/cloud-init-build-rpm/1/console I'll do the testing of the rpm's tomorrow. | 23:09 |
larsks | rharper: I can take a closer look later this evening; kids go to sleep by 9pm (us/eastern). | 23:09 |
rharper | larsks: sure, thanks | 23:09 |
rharper | powersj: \o/ | 23:09 |
larsks | Will you be around then? And/or are you using a spec file different from what smoser posted? | 23:10 |
rharper | I'll be out though | 23:11 |
smoser | larsks, well ^ is in trunk now. | 23:11 |
smoser | you can 'make srpm' | 23:11 |
larsks | Cool. | 23:11 |
larsks | Will do. | 23:11 |
rharper | PYVER=python2 make srpm | 23:11 |
rharper | one of the two | 23:11 |
smoser | it seems odd to me that one would need from cent6 to cent7 people would have to change every package that had systemd scripts | 23:12 |
smoser | i guess maybe over time in fedora that just happened. | 23:12 |
smoser | blackboxsw, | 23:39 |
smoser | return json.dumps(data, indent=1, sort_keys=True, | 23:39 |
smoser | separators=(',', ': ')).encode('utf-8') | 23:39 |
smoser | format the json with that json.dumps | 23:39 |
blackboxsw | thanks | 23:41 |
rharper | smoser: larsks: this looks to work http://paste.ubuntu.com/24811524/ | 23:49 |
rharper | re: specfile updates | 23:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!