Attachment 649998 Details for Bug 879187 – IRC log extract from when we discussed applying this patch (for forensic purposes)

IRC log extract from when we discussed applying this patch (for forensic purposes)

file_879187.txt (text/plain), 3.47 KB, created by Adam Williamson on 2012-11-22 19:05:30 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Adam Williamson

Created: 2012-11-22 19:05:30 UTC

Size: 3.47 KB

patch

obsolete

>20-11-2012 11:29:34 < bcl!~bcl@neil.brianlane.com: jlk: you left an import pdb in that patch for 876897
>20-11-2012 11:29:49 < jlk!~jkeating@fedora/ender: haha
>20-11-2012 11:29:50 < jlk!~jkeating@fedora/ender: whoops
>20-11-2012 11:30:01 < bcl!~bcl@neil.brianlane.com: brown m&m's, right?
>20-11-2012 11:30:06 < jlk!~jkeating@fedora/ender: sure, that's it
>20-11-2012 11:30:30 < jlk!~jkeating@fedora/ender: fixed locally
>20-11-2012 11:32:09 < bcl!~bcl@neil.brianlane.com: ack otherwise. 
>20-11-2012 11:32:29 < jlk!~jkeating@fedora/ender: pushing / picking
>20-11-2012 11:32:31 < jlk!~jkeating@fedora/ender: pushing
>20-11-2012 11:33:11 < bcl!~bcl@neil.brianlane.com: yeah, that's a beta nth to pick it to f18-beta2-branch as well.
>20-11-2012 11:33:11 < jlk!~jkeating@fedora/ender: vunderbar.  Conflicts
>20-11-2012 11:33:17 < bcl!~bcl@neil.brianlane.com: whee :)
>20-11-2012 11:33:56 < jlk!~jkeating@fedora/ender: oh right, this also requires a commit I did that added some constants
>20-11-2012 11:34:06 < dlehman!~dlehman@67.58.72.173: adamw: apparently it's easier to hit 877871 than I thought. if you mark an existing LV whose PVs are encrypted for reformat you can/will hit it
>20-11-2012 11:34:36 < jlk!~jkeating@fedora/ender: We'll have to cherry pick over 98538bb89f3c82babf86dcaf2936ba08a67fd08d which fixes nfsiso stuff too
>20-11-2012 11:34:47 < dlehman!~dlehman@67.58.72.173: adamw: but I can add a trivial fix for that case to the other bug's patch. the ui-twiddling can be handled separately post-beta.
>20-11-2012 11:34:53 < bcl!~bcl@neil.brianlane.com: if it depends on it that's fine with me.
>20-11-2012 11:35:03 < jlk!~jkeating@fedora/ender: that also resolves the conflict :)
>20-11-2012 11:35:15 < tflink!~tflink@redhat/tflink: wwoods: can you make an update for the new fedup-dracut builds if they're ready? The new lorax is likely to be used for the next TC/RC and I'd like to try using a more official repo for fedup testing
>20-11-2012 11:36:38 < jlk!~jkeating@fedora/ender: is smoke 22 built from the beta2 branch?
>20-11-2012 11:37:07 < tflink!~tflink@redhat/tflink: jlk: not sure. I pulled in the anaconda-18.30-1 scratch build that bcl did last night
>20-11-2012 11:37:21 < tflink!~tflink@redhat/tflink: if that was built from the beta2 branch, then yes
>20-11-2012 11:37:34 < jlk!~jkeating@fedora/ender: bcl: remember that question about confusing others?  :)
>20-11-2012 11:37:49 < dlehman!~dlehman@67.58.72.173: adamw: I know this probably seems like a mess, but when you have 8 patches in the works you can't always test them in isolation from each other and then again for every possible subset
>20-11-2012 11:37:53 < bcl!~bcl@neil.brianlane.com: tflink: that's from master. beta builds will now be 18.29.X
>20-11-2012 11:38:17 < bcl!~bcl@neil.brianlane.com: so smoke22 is much bettter than beta :)
>20-11-2012 11:38:24 < tflink!~tflink@redhat/tflink: bcl: oh. ok. I probably shouldn't have pulled that in to the latest smoke build then
>20-11-2012 11:38:35 < * tflink!~tflink@redhat/tflink learned something new today
>20-11-2012 11:38:40 < bcl!~bcl@neil.brianlane.com: hey, makes me happy :)
>20-11-2012 11:38:48 < bcl!~bcl@neil.brianlane.com: sorry for the confusion though.
>20-11-2012 11:38:56 < tflink!~tflink@redhat/tflink: I was wondering why the numbering scheme changed all of a sudden
>20-11-2012 11:39:11 < bcl!~bcl@neil.brianlane.com: The goal is to not get to a point where the stuff on master has had no testing.
>20-11-2012 11:49:17 > adamw: dlehman: sure, i get that. go ahead and put that fix in.

Actions: View

Attachments on bug 879187: 649611 | 649612 | 649613 | 649614 | 649615 | 649616 | 649617 | 649618 | 649619 | 649620 | 649621 | 649622 | 649623 | 649624 | 649625 | 649795 | 649796 | 649798 | 649799 | 649802 | 649803 | 649804 | 649805 | 649998 | 659293 | 659296