jamespagecpaelzer__: when you're around - both of those MIR's are required for uploads already in disco-proposed (nova and neutron) (also commented on bug)05:06
jamespagethanks for the reviews05:07
cpaelzer__jamespage: ok, good to know05:36
cpaelzer__jamespage: are there others still open blocking you?05:36
=== cpaelzer__ is now known as cpaelzer
jamespagecpaelzer: http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses_by_team.html#ubuntu-openstack05:36
jamespagehttps://bugs.launchpad.net/ubuntu/+source/websockify/+bug/1108935 is as well05:36
ubottuLaunchpad bug 1108935 in websockify (Ubuntu) "[MIR] websockify, spice-html5" [High,Confirmed]05:36
cpaelzerthere is more history to be read on that one for sure ...05:43
cpaelzerjamespage: this actually is ok already IMHO05:45
cpaelzerin comment #8 you got the MIR ACK05:45
cpaelzerin comment #11 and #12 you got the conditional security ack05:46
cpaelzerand you later adressed these conditions05:46
jamespagecpaelzer: yup that summarises it nicely!05:46
cpaelzerOTOH this review stretched 6 years now :-/05:47
cpaelzerwell the active part 2.5 years05:47
cpaelzerGiven that former versions where seucrity acked I'll do a full recheck with that in mind05:47
cpaelzermost likely that will unblock it for you then05:47
cpaelzerbut I had no breakfast yet, so first things first :-)05:48
jamespagecpaelzer: breakfast sounds like the priority to me05:48
cpaelzerjamespage: which openstack exactly does (or would) add the websockify dependency?06:52
cpaelzerif not yet in a d/control somewhere which entry for depends/recommends did you have in mind?06:52
jamespagecpaelzer: coreycb added the depends for this cycle I think06:52
jamespagethe version in proposed def runtime depends on it06:52
cpaelzerpython3-websockify I see in nova06:56
cpaelzerfrom python3-nova06:57
cpaelzerI was concerned since e.g. websockify binary itself still depends on python2 bits06:57
cpaelzerlet me make sure in a contianer it does not pull in py206:57
cpaelzerjamespage: websockify itself is good07:03
cpaelzerjamespage: but spice-html5 needs a check now, doing that next07:04
jamespagecpaelzer: looking at it we've not seeded nova-spiceproxy07:09
jamespageso spice-html5 is not being pulled to main07:09
cpaelzerjamespage: could you put that as an answer to https://bugs.launchpad.net/ubuntu/+source/websockify/+bug/1108935/comments/24 ?07:18
ubottuLaunchpad bug 1108935 in spice-html5 (Ubuntu) "[MIR] websockify, spice-html5" [High,Confirmed]07:18
jamespageI have07:19
cpaelzerjamespage: you should at some point add nova-spiceproxy as a suggest to some other package in there07:20
cpaelzeror people won't even know07:20
cpaelzerjamespage: so should I abort the spice-html5 review as you do't need it?07:21
cpaelzerjamespage: or do you need it to actually make use of that function, yet only the dependency to pull it in is missing?07:21
cpaelzerok thanks, going on then07:21
jamespagebut I'll really need a release team +1 to add that as a seeded package07:21
cpaelzerbut no matter how early or late that review needs to be done in case more follow on actionsare identified07:22
jamespagecpaelzer: lets push the spice-html5 to next cycle - sounds like we need to re-review anyway07:31
jamespagethankyou for your review...07:31
cpaelzerjamespage: you have an ack under constraints for spice-html507:51
cpaelzerjamespage: doing it next cycle seems wise, please put the tasks I outlined on your teams list07:51
cpaelzeryou can now self-resolve this to be promotable to main next cycle07:52
=== cpaelzer__ is now known as cpaelzer
=== ricab is now known as ricab|lunch
Kolargol00Hi there! Can someone help me by sponsoring/reviewing a SRU and a backport requests?13:25
Kolargol00https://launchpad.net/bugs/1822069, https://launchpad.net/bugs/182188813:25
ubottuLaunchpad bug 1822069 in xmltooling (Ubuntu) "SRU: Shibboleth SPv3 for bionic" [Undecided,New]13:25
ubottuLaunchpad bug 1821888 in Xenial Backports "Please backport xerces-c 3.2.0+debian-2 (universe) from bionic" [Undecided,Confirmed]13:25
LeoBrasHello, I want to fix a bug on libvirt* that is already fixed on mainline (it's a backport to bionic). Where is the right place to get the source code?13:55
LeoBrasFor me, it seems that getting the code from 'apt-get source' is different from 'pull-lp-source' even if they are both from bionic14:14
LeoBrasAnd both are different from the git repo14:17
LeoBraswhich one is the right one to fix bugs for?14:18
=== ricab|lunch is now known as ricab
EickmeyerLeoBras: https://launchpad.net/ubuntu/+source/libvirt14:25
EickmeyerIf you file the bug and have a fix, include the patch.14:25
LeoBrasIt points to the git repo, i was right to patch towards this. I need to test the patch first, but building seems to fail when using 'fakeroot debian/rules binary'14:26
LeoBras(I am trying to build for ppc64el)14:27
LeoBras(i had done a build-dep before building)14:27
EickmeyerLeoBras: What I linked is where the bugs get reported and patches get included. That's where it's done for Ubuntu. If it's fixed upstream and hasn't been merged yet, then that's up to the core developers since the package is in the "Main" repo.14:29
LeoBrasWell, ok, but I can't build even the repository as is using 'fakeroot debian/rules binary'14:31
LeoBrashow am I supposed to test my patch?14:31
EickmeyerLeoBras: In order to do that, you need to be able to package it for Ubuntu and test it that way.14:32
LeoBraswhat does it mean 'to be able to package it for Ubuntu' ?14:32
Eickmeyer"Debian/rules" isn't a binary, it's part of the packaging. fakeroot is a tool used for packaging.14:33
LeoBraswhat should I use to build the package so?14:36
EickmeyerLeoBras: The instructions are what I just linked you to.14:37
EickmeyerI'm afraid I can't help you any further.14:37
LeoBrason this guide, it says to get the source code using 'pull-lp-source', and that source differs from the source on https://code.launchpad.net/ubuntu/+source/libvirt14:38
LeoBrasis ther e a reason for it to be different?14:39
cpaelzerLeoBras: you need to make sure to get the right version(s)14:49
cpaelzerhttps://code.launchpad.net/~usd-import-team/ubuntu/+source/libvirt/+git/libvirt/+ref/ubuntu/bionic-devel should match what "pull-lp-source libvirt bionic" gives you right now14:49
cpaelzerLeoBras: or you can use https://snapcraft.io/git-ubuntu and just run "git ubuntu clone libvirt"14:51
cpaelzerLeoBras: could you sent me a sneak peak to the upstream change please?14:51
LeoBrascpaelzer, oh I cloned https://code.launchpad.net/ubuntu/+source/libvirt and the only bionic branch was origin/ubuntu/bionic-4.014:53
LeoBrascpaelzer, about building the change locally on a ubuntu package, how to do it?15:00
LeoBrascpaelzer, oops, sorry, I accidentally cloned git://git.launchpad.net/~libvirt-maintainers/ubuntu/+source/libvirt my mistake15:02
LeoBrascpaelzer, oh the upstream commit is 55ce656415:13
LeoBrascpaelzer, The bug is being migrated from our bugzilla to launchpad soon15:14
LeoBrascpaelzer, it seems the patch is already applied, but the git history seems to not describe this single patch15:21
wxlcyphermox: @tsimonq2 suggested i get in touch with you. as you know, lubuntu uses calamares installer. for encryption, it uses the latest cryptsetup, which defaults to luks2.. which is incompatible with grub2. apparently there's a compile-time option to set the default version to luks1. as i read it, this would not eliminate the abilitity of others to use luks2 (although it's a bit impractical given the17:32
wxlstate of grub). do you think we could make this the default?17:32
pedahzurTrying to walk https://bugs.launchpad.net/ubuntu/+source/adcli/+bug/1821242 through the fix process, and am reading https://wiki.ubuntu.com/StableReleaseUpdates. Is there a "First timer HOWTO doc" somewhere out there?  This is all really new to me. :)18:48
ubottuLaunchpad bug 1821242 in adcli (Ubuntu Bionic) "adcli delete dies with free(): invalid pointer" [High,Triaged]18:48
rbasakpedahzur: thank you for working on that. I'm sorry the documentation isn't any better. You're in the right place to ask though, and I and others would be happy to walk you through it here.19:07
rbasakIt's late in the evening for me and I'm about to disappear though, sorry.19:07
pedahzurIt's not *terribly* time crucial, as we have a local build with the fix, but I would like to see this fixed in the official package. I'll hit you up Monday morning (my time) and we can see if we can walk through it.19:08
pedahzurrbasak: Hopefully that will hit your "afternoon" and we have an hour or two of overlap. :)19:09
rbasakSure. Thanks!19:09
=== themill_ is now known as themill
=== DalekSec_ is now known as DalekSec
=== Wryhder is now known as Lucas_Gray
=== albert23` is now known as albert23
=== balloons8 is now known as balloons

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!