| 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!