*** SumitNaiksatam has quit IRC | 00:27 | |
bedis | Hey ! | 00:28 |
---|---|---|
crc32 | hello | 00:39 |
openstackgerrit | Li Yingjun proposed openstack/neutron-lbaas: Persistence lbaas http cookie https://review.openstack.org/143400 | 00:41 |
*** rm_work is now known as rm_work|away | 00:43 | |
*** ajmiller has quit IRC | 00:43 | |
*** hichtakk has quit IRC | 00:47 | |
*** hichtakk has joined #openstack-lbaas | 00:48 | |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Partially-Implements: blueprint lbaas-tls Change-Id: I0805b6dae0dcc311b176e3017f0218e30b7d81f5 https://review.openstack.org/142915 | 00:48 |
dougwig | hi bedis! | 00:52 |
dougwig | you back in france? | 00:53 |
crc32 | No distributions matching the version for SQLAlchemy<=0.8.99,<=0.9.99,>=0.8.4,>=0.9.7 (from neutron-lbaas==2015.1.dev7) <-- Not sure why I can't run tox tests now. | 00:57 |
*** hichtakk has quit IRC | 00:58 | |
*** hichtakk has joined #openstack-lbaas | 00:58 | |
sbalukoff | Ok crc32, I just sent you a key / cert to play with, as well as instructions for generating more like this (ie. x509v3 extensions for both client and server usage) | 00:59 |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Partially-Implements: blueprint lbaas-tls Change-Id: I0805b6dae0dcc311b176e3017f0218e30b7d81f5 https://review.openstack.org/142915 | 01:01 |
crc32 | sbalakuoff: Ok thank you. I'm researching x509Exts next. I just hope enough of the low level stuff is exposed to pyOpenSSL. | 01:05 |
crc32 | Good night. | 01:06 |
crc32 | for now. | 01:06 |
sbalukoff | Have a good one | 01:06 |
*** crc32 has quit IRC | 01:07 | |
*** xgerman has quit IRC | 01:17 | |
*** hichtakk has quit IRC | 01:18 | |
*** hichtakk has joined #openstack-lbaas | 01:18 | |
*** mrwangmaster has joined #openstack-lbaas | 01:32 | |
*** markmcclain has quit IRC | 01:59 | |
*** markmcclain has joined #openstack-lbaas | 02:03 | |
*** blogan_ has joined #openstack-lbaas | 02:17 | |
*** SumitNaiksatam has joined #openstack-lbaas | 02:51 | |
*** mlavalle has quit IRC | 03:11 | |
*** markmcclain has quit IRC | 03:20 | |
*** rm_work|away is now known as rm_work | 03:24 | |
mrwangmaster | I'm new to openstack, but being asked to spin up on it and work on a lbaas deployment... curious if anyone has thoughts on if any training courses are worth the time? or best to just dig into docs & start building a lab env? | 03:24 |
*** hichtakk has quit IRC | 03:33 | |
*** hichtakk has joined #openstack-lbaas | 03:33 | |
rm_work | hmm I have no idea how SQLAlchemy<=0.8.99,<=0.9.99,>=0.8.4,>=0.9.7 would ever work | 03:36 |
rm_work | doesn't that exclude... everything/ | 03:36 |
rm_work | mrwangmaster: randomly, does #archive mean anything to you? | 03:45 |
mrwangmaster | rm_work: can't say as I was super active in such a channel. | 03:47 |
rm_work | ok, just curious | 03:47 |
rm_work | mrwangmaster: I would say try spinning up a devstack instance, and check that out | 03:47 |
rm_work | somewhere there is a howto, let me see... | 03:47 |
rm_work | I don't know if this is up to date: https://wiki.openstack.org/wiki/Neutron/LBaaS/HowToRun | 03:48 |
mrwangmaster | rm_work: yeah... that's my normal tact; however mgmr is asking, if someone said "oh yeah xyz training is great" then maybe, otherwise thinking it's best to just dive into things. | 03:48 |
*** fnaval has quit IRC | 04:20 | |
*** fnaval has joined #openstack-lbaas | 04:29 | |
*** blogan_ has quit IRC | 04:50 | |
*** hichtakk has quit IRC | 04:51 | |
*** hichtakk has joined #openstack-lbaas | 05:00 | |
*** hichtakk has quit IRC | 05:03 | |
*** jschwarz has joined #openstack-lbaas | 05:55 | |
*** hichtakk has joined #openstack-lbaas | 06:15 | |
*** sbfox has joined #openstack-lbaas | 06:19 | |
*** sbfox has quit IRC | 06:43 | |
*** apuimedo has joined #openstack-lbaas | 06:55 | |
*** fnaval has quit IRC | 07:00 | |
*** fnaval has joined #openstack-lbaas | 07:00 | |
*** fnaval has quit IRC | 07:05 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 07:32 |
*** kobis has joined #openstack-lbaas | 07:38 | |
*** hichtakk has quit IRC | 07:41 | |
*** sbfox has joined #openstack-lbaas | 07:43 | |
*** hichtakk has joined #openstack-lbaas | 08:02 | |
*** sbfox has quit IRC | 08:15 | |
*** hichtakk has quit IRC | 08:23 | |
*** hichtakk has joined #openstack-lbaas | 08:46 | |
*** woodster_ has quit IRC | 09:00 | |
*** hichtakk has quit IRC | 09:17 | |
openstackgerrit | Adam Harwell proposed stackforge/octavia: Update TLS Specification https://review.openstack.org/142572 | 09:43 |
rm_work | sbalukoff: updated, actually had done it at the hackathon but forgot to commit/review it T_T | 09:43 |
rm_work | ^^ | 09:43 |
openstackgerrit | Adam Harwell proposed stackforge/octavia: Update certificate generator implementations https://review.openstack.org/142629 | 09:45 |
rm_work | ^^ rebased | 09:57 |
*** rm_work is now known as rm_work|away | 11:07 | |
*** enikanorov_ has quit IRC | 11:30 | |
*** jschwarz has quit IRC | 11:31 | |
*** Putns has quit IRC | 12:24 | |
*** rohara1 has quit IRC | 12:24 | |
*** dougwig has quit IRC | 12:24 | |
*** BeardyMcBeards has quit IRC | 12:24 | |
*** rm_work|away has quit IRC | 12:24 | |
*** davidlenwell has quit IRC | 12:24 | |
*** clev has quit IRC | 12:24 | |
*** apuimedo has quit IRC | 12:24 | |
*** redrobot_away has quit IRC | 12:24 | |
*** Putns has joined #openstack-lbaas | 12:26 | |
*** rohara1 has joined #openstack-lbaas | 12:26 | |
*** dougwig has joined #openstack-lbaas | 12:26 | |
*** BeardyMcBeards has joined #openstack-lbaas | 12:26 | |
*** rm_work|away has joined #openstack-lbaas | 12:26 | |
*** davidlenwell has joined #openstack-lbaas | 12:27 | |
*** clev has joined #openstack-lbaas | 12:27 | |
*** apuimedo has joined #openstack-lbaas | 12:28 | |
*** redrobot_away has joined #openstack-lbaas | 12:28 | |
*** fnaval has joined #openstack-lbaas | 12:57 | |
*** TrevorV_ has joined #openstack-lbaas | 14:43 | |
*** apuimedo has quit IRC | 14:47 | |
*** enikanorov has joined #openstack-lbaas | 14:50 | |
sbalukoff | Morning! | 14:58 |
a2hill | Mornin' | 15:07 |
*** fnaval has quit IRC | 15:07 | |
*** fnaval has joined #openstack-lbaas | 15:08 | |
*** fnaval has quit IRC | 15:13 | |
*** ajmiller has joined #openstack-lbaas | 15:25 | |
*** fnaval has joined #openstack-lbaas | 15:33 | |
*** ajmiller_ has joined #openstack-lbaas | 15:49 | |
*** ajmiller__ has joined #openstack-lbaas | 15:52 | |
*** ajmiller has quit IRC | 15:53 | |
*** ajmiller_ has quit IRC | 15:55 | |
*** ajmiller__ has quit IRC | 15:58 | |
*** xgerman has joined #openstack-lbaas | 16:03 | |
*** markmcclain has joined #openstack-lbaas | 16:04 | |
*** ajmiller has joined #openstack-lbaas | 16:08 | |
*** markmcclain has quit IRC | 16:48 | |
*** mlavalle has joined #openstack-lbaas | 17:15 | |
*** kobis has quit IRC | 17:20 | |
dougwig | morning | 17:29 |
dougwig | mrwangmaster: openstack is somewhat of a "some assembly required" piece of software, so whether the training makes sense depends on your level/desire to tinker. | 17:33 |
*** openstackgerrit has quit IRC | 17:34 | |
*** sbfox has joined #openstack-lbaas | 17:34 | |
*** openstackgerrit has joined #openstack-lbaas | 17:35 | |
*** zeus_ has joined #openstack-lbaas | 17:35 | |
*** zeus_ is now known as zeus | 17:35 | |
*** zeus has joined #openstack-lbaas | 17:35 | |
*** hichtakk has joined #openstack-lbaas | 17:41 | |
mrwangmaster | dougwig: yeah, understandable. I've already let $bossman know that I don't consider the training to be a total waste of time, and would go, but that I'll likely be better off spending the time in my own lab & reading docs. | 17:45 |
*** SumitNaiksatam has quit IRC | 17:45 | |
dougwig | i was ready to throw my entire rig out the window many times. :) | 17:52 |
mrwangmaster | hehe, setting up a lab is sorta on the agenda today | 17:56 |
mrwangmaster | but had to deal with a customer this AM, so sched is off a bit. | 17:56 |
*** sbfox has quit IRC | 18:02 | |
*** SumitNaiksatam has joined #openstack-lbaas | 18:09 | |
*** sbfox has joined #openstack-lbaas | 18:11 | |
*** sbfox has quit IRC | 18:38 | |
*** sbfox has joined #openstack-lbaas | 18:48 | |
*** markmcclain has joined #openstack-lbaas | 18:52 | |
*** markmcclain has quit IRC | 19:01 | |
*** crc32 has joined #openstack-lbaas | 19:20 | |
crc32 | https://review.openstack.org/#/c/142915/ <-- Trying to figure out why jenkins is puking on SQLAlchemy and PrettyTable during testing. | 19:33 |
a2hill | What are the errors? | 19:34 |
a2hill | oh, jenkins, was reading the comment and this | 19:36 |
a2hill | No distributions matching the version for SQLAlchemy<=0.8.99,<=0.9.99,>=0.8.4,>=0.9.7 (from neutron-lbaas==2015.1.dev7) | 19:38 |
a2hill | yet: 2014-12-23 07:34:42.069 | SQLAlchemy==0.9.8 | 19:39 |
a2hill | Having issues reaching repo? | 19:40 |
a2hill | Some externally hosted files were ignored as access to them may be unreliable (use --allow-external to allow) | 19:40 |
*** woodster_ has joined #openstack-lbaas | 19:43 | |
crc32 | yea not sure. dougwig suggested this was an infra structure issue that was effecting all jenkins builds. But when I looked at zuul.openstack.org I only see red on the lbaas projects. | 19:44 |
crc32 | the main ones I was seeing was "prettable<0.7" | 19:45 |
a2hill | :/ | 19:45 |
a2hill | in your commit i only see it complaining about sqlalchemy, eventhough the correct version is in the env | 19:46 |
crc32 | "SQLAlchemy<=0.8.99,<=0.9.99,>=0.8.4,>=0.9.7" This of course would be impossable. How can you have a number greater then 9.7 yet less then 8.99. Where did this come from? | 19:50 |
a2hill | :/ | 19:51 |
a2hill | aliens | 19:51 |
crc32 | Are the comma's supposed to be treated as bitwise OR operations? | 19:52 |
*** sbfox has quit IRC | 19:52 | |
crc32 | Any ways I think its a pip version bork. Cause on my machine that requirment installs SQLAchemy 0.9.8 | 19:59 |
crc32 | yet in jenkins it diliberatly excludes it. | 20:00 |
dougwig | prettytable requirement got busted by a change in pip. it's fixed now. | 20:04 |
openstackgerrit | German Eichberger proposed stackforge/octavia: taskflows to create, delete amphora https://review.openstack.org/142569 | 20:19 |
*** xgerman has quit IRC | 20:27 | |
*** xgerman has joined #openstack-lbaas | 20:27 | |
crc32 | dougwig: It looks like the build got further along in jenkins. but now I'm hunting through a 1.2 Meg file looking for what tempest test allegedly broke. http://logs.openstack.org/15/142915/6/check/check-tempest-dsvm-neutron-pg-full/5586d4c/console.html | 20:30 |
crc32 | What is the stradegy for looking through this needle in a hay stack. I though I could look for "Error" but have the tests have names like "test_for_error" or something. | 20:31 |
crc32 | I also see "SSH could not connect type errors" but I don't know if this represent inconclusive tests or if these represent why the commit is being rejected. :/ | 20:32 |
crc32 | Whats your grep stradegy? | 20:32 |
*** sbfox has joined #openstack-lbaas | 20:32 | |
xgerman | a2hill: https://review.openstack.org/#/c/142569/ this has some of the task flow code and even starts a nova vm | 20:33 |
crc32 | its too bad they don't have keywords like "commit_rejecting_ERROR" to search for. | 20:33 |
a2hill | was looking at that xgerman | 20:33 |
xgerman | awesome -- | 20:34 |
a2hill | This doesnt handle whats returned from the driver though does it? | 20:34 |
a2hill | trying to find the link since this just lives in the controller package | 20:34 |
xgerman | it will onc ethere is code | 20:35 |
a2hill | ah | 20:35 |
xgerman | if a "task/flow" needs to be in the driver I put that in the comments to the task | 20:35 |
xgerman | but all those flows will be in the controller and then call out to to the driver to get the specific stuff | 20:36 |
a2hill | gotcha | 20:36 |
xgerman | the idea was to stubb that out so we can start a nova vm | 20:36 |
xgerman | doug even wnated to have code to ssh in and start haproxy | 20:37 |
a2hill | ah, thought thats what the amphora api was going to handle? | 20:37 |
a2hill | though, is that just going to call flows also? | 20:37 |
xgerman | yep, it will | 20:37 |
xgerman | but we wanted to have something to play with | 20:37 |
a2hill | ah, gotcha | 20:37 |
xgerman | so I left the ssh stuff out :-) | 20:37 |
a2hill | :) | 20:37 |
xgerman | controller.api_server.api_server | 20:38 |
xgerman | is a command line executable so you can run some flow | 20:38 |
a2hill | nice | 20:38 |
xgerman | yeah, one day it will listen on some queue | 20:38 |
a2hill | :P | 20:38 |
xgerman | so if you add the amphora driver you can wire up the flows and run that to see if it works | 20:39 |
*** TrevorV_ has quit IRC | 20:39 | |
a2hill | perfect, going to dabble in this. Thanks xgerman! | 20:40 |
xgerman | cool - | 20:40 |
xgerman | I assume we will keep iterating a lot on that -- but at least we have some code :-) dougwig: we can fire up a nova vm... | 20:42 |
a2hill | Think theres going to be any major changes to the interface? | 20:42 |
xgerman | no, I think it's pretty syable | 20:43 |
xgerman | the controller flow will change as the pieces (amphora driver, network driver, etc.) come online | 20:44 |
a2hill | indeed | 20:46 |
*** markmcclain has joined #openstack-lbaas | 20:47 | |
*** sbfox has quit IRC | 20:47 | |
xgerman | one big outcome form the hackathon was that we now have an idea how the pieces fit together and that was my attempt to put it in code :-) | 20:48 |
crc32 | Are these SSH connection errors whats killing my tests in CR https://review.openstack.org/#/c/142915 | 20:52 |
crc32 | ok so some one removed SQLAlchemy <=0.8 from openstack/requiremtns/global-requirements.txt so I'll assume these old versions of SQL alchemy are not supported anymore. I'm removing it. | 20:58 |
crc32 | -# The SQLA requirement looks weird, but it's here for a reason. The | 20:58 |
crc32 | -# version of pip shipped with distros will treat 0.9b* as < 0.8 if it's | 20:58 |
crc32 | -# anytime uploaded to PyPI, so the "clear" version of this requirement | 20:58 |
crc32 | -# is potentially broken. The fix for this isn't until pip 1.4.1, which | 20:58 |
crc32 | -# is way more current than what most distros have. | 20:58 |
crc32 | -SQLAlchemy>=0.8.4,<=0.8.99,>=0.9.7,<=0.9.99 | 20:58 |
crc32 | +SQLAlchemy>=0.9.7,<=0.9.99 | 20:58 |
crc32 | the 0.8 Dependencies were removed on December 7th. So I'll mimic that line in neutron-lbaas | 20:59 |
*** sbfox has joined #openstack-lbaas | 20:59 | |
*** markmcclain1 has joined #openstack-lbaas | 21:00 | |
*** markmcclain has quit IRC | 21:00 | |
*** markmcclain1 has quit IRC | 21:02 | |
*** markmcclain has joined #openstack-lbaas | 21:02 | |
*** markmcclain has quit IRC | 21:03 | |
xgerman | I was just looking at https://github.com/dreamhost/akanda-rug/blob/master/akanda/rug/vm_manager.py which marmcclain proposed and now he quits :-) | 21:03 |
crc32 | still doesn't explain the SSH borkage though. | 21:05 |
*** rm_work|away is now known as rm_work | 21:08 | |
*** xgerman has quit IRC | 21:08 | |
rm_work | crc32: yeah i don't know why our requirements weren't updated already to match | 21:08 |
rm_work | but that should have done in | 21:08 |
rm_work | *done it | 21:08 |
crc32 | well I'm attempting to do it on my commit. | 21:09 |
crc32 | in our tox.ini file for neutron-lbaas I see this | 21:12 |
crc32 | deps = -egit+https://git.openstack.org/openstack/neutron#egg=neutron | 21:12 |
crc32 | -r{toxinidir}/requirements.txt | 21:12 |
crc32 | -r{toxinidir}/test-requirements.txt | 21:12 |
rm_work | yeah that looks right I think | 21:13 |
crc32 | yet an explicit commit comments out the line # -e git+https://git.openstack.org/openstack/neutron#egg=neutron | 21:13 |
rm_work | err which one | 21:13 |
crc32 | on master. What is the real intent? | 21:14 |
crc32 | It looks like doug removed it from requirements.txt but added it to tox.ini | 21:14 |
rm_work | yes | 21:14 |
crc32 | eb41ab78b87f1589fada5fca0c9189fa361d6289 | 21:14 |
rm_work | I thought we talked about this earlier | 21:14 |
rm_work | we don't want to have neutron as a dependency, because in deployments it will be handled manually outside of our scope | 21:15 |
rm_work | but we need it for test runs to we install it directly with tox | 21:15 |
crc32 | No you said it was removed from requirments. Our discussion was we needed to install nuetron manually. There was no mention of it being installed via tox.ini. I'm trying to get through this requirments hell right now is all rm_work. I'll assume its supposed to be in tox.ini. | 21:15 |
rm_work | yeah, tox handles testing, so it must be included there for the tests to pass | 21:16 |
rm_work | it goes into the tox environments so it should not affect anything else on the system | 21:16 |
crc32 | great thats all I needed to hear. I was freaking out wondering how neutron was being pulled in when it wasn't in requirments.txt | 21:16 |
crc32 | that is all. | 21:16 |
rm_work | cool | 21:17 |
rm_work | brb | 21:17 |
crc32 | at anyrate I'm hoping this cleans up the SQLAlchemy bork but I'm not sure about the SSH bork. | 21:17 |
*** rm_work is now known as rm_work|away | 21:17 | |
dougwig | sweet on the nova bouts. | 21:20 |
dougwig | boot | 21:20 |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Partially-Implements: blueprint lbaas-tls Change-Id: I0805b6dae0dcc311b176e3017f0218e30b7d81f5 https://review.openstack.org/142915 | 21:20 |
*** rm_work|away is now known as rm_work | 21:20 | |
dougwig | crc32: i usually scroll to the bottom and head upwards. search for FAILED | 21:20 |
rm_work | crc32: also, you need to fix your commit message :P I posted a comment with the correct syntax on your change | 21:21 |
rm_work | also I think doug fixed it too but it got overwritten | 21:21 |
crc32 | I'm not even changing the commit message. | 21:21 |
rm_work | crc32: right, you *need to* change it | 21:21 |
crc32 | dougwig changed it in a patchset. Does that mean its reverted now? | 21:22 |
crc32 | to mine? | 21:22 |
rm_work | yeah, since you didn't pull down the newest before you made your next review on top | 21:22 |
rm_work | not a huge deal, but will need to be fixed eventually | 21:23 |
*** sbfox has quit IRC | 21:23 | |
rm_work | just do it along with your next change | 21:24 |
*** zeus has quit IRC | 21:25 | |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 21:27 |
rm_work | crc32: I'll take a look at the logs when the tests finish running this time and see if I can spot anything | 21:29 |
*** xgerman has joined #openstack-lbaas | 21:29 | |
rm_work | crc32: so far everything is looking good | 21:37 |
*** xgerman has quit IRC | 21:41 | |
crc32 | I'm waiting for SSH to fail. | 21:47 |
rm_work | when does that usually happen? | 21:48 |
rm_work | most of the gates have already passed | 21:48 |
crc32 | not sure but it takes like 30mins or so. | 21:53 |
crc32 | hte ssh ones only happened once though. | 21:53 |
rm_work | you watching it on http://status.openstack.org/zuul/ ? | 21:54 |
*** sbfox has joined #openstack-lbaas | 21:55 | |
crc32 | was watching on zuul.openstack.org | 22:02 |
rm_work | ah lol they have two of the same thing with different CSS <_< | 22:03 |
rm_work | so anyway yeah | 22:09 |
rm_work | are they at work today? | 22:09 |
rm_work | crc32: ^^ | 22:10 |
rm_work | TrevorV / jorgem | 22:10 |
rm_work | crc32: we ended up in that channel because i was saying something to the barbican people and then started talking to you and forgot what channel I was in :P | 22:11 |
rm_work | since we're both in both | 22:11 |
a2hill | i thought jorge was out for holidays also? | 22:12 |
rm_work | ah he might be | 22:13 |
rm_work | a2hill: you have a core openstack commit in, right? | 22:13 |
a2hill | Not sure | 22:13 |
a2hill | :/ | 22:13 |
rm_work | ah, you'll have the HAProxy namespace driver stuff soon I think | 22:13 |
a2hill | Yea | 22:13 |
rm_work | since that was what you were working on at some point that got stalled | 22:13 |
a2hill | ? | 22:13 |
rm_work | (with the split and such) | 22:14 |
rm_work | all of our actual neutron-lbaas stuff was stalled waiting on split resolution, and we moved to Octavia tasks | 22:14 |
a2hill | the meeting in feb if for neutron/lbaas right | 22:14 |
a2hill | well yea | 22:14 |
rm_work | yes | 22:14 |
a2hill | was curious why you were asking though | 22:14 |
a2hill | oh | 22:15 |
rm_work | because carlos has an extra really really easy one-liner that should merge easily | 22:15 |
rm_work | so someone who needs a commit could take it | 22:15 |
crc32 | So your giving him my commit? | 22:15 |
rm_work | I thought maybe jorge or trevor since they have been primarily on Octavia stuff | 22:15 |
rm_work | crc32: you have a commit (the TLS stuff) | 22:15 |
a2hill | was going to say i had the non-tls template in thats merged but think thats under another user now :/ | 22:15 |
rm_work | I'm talking about the requirements.ini change | 22:15 |
crc32 | ok. Give it to him. | 22:15 |
crc32 | requirements.txt | 22:15 |
rm_work | lol yeah that | 22:16 |
crc32 | so do I have a commit already? | 22:16 |
rm_work | i mean, if neither of them are in, you can just split it out and take it yourself, more stuff will probably come up I guess | 22:16 |
rm_work | crc32: i don't know actually, but the TLS one will count | 22:16 |
a2hill | some of the code in here was mine | 22:16 |
a2hill | https://review.openstack.org/#/c/141247/ | 22:16 |
crc32 | I'll just check out master again and rebase? | 22:16 |
a2hill | why concerned with having a core commit? | 22:17 |
rm_work | a2hill: for ATC codes | 22:17 |
a2hill | AH | 22:17 |
a2hill | ;) | 22:17 |
a2hill | if we get to go | 22:17 |
rm_work | assuming we actually have travel budget yeah <_< | 22:17 |
a2hill | :P | 22:17 |
a2hill | if anything i want to make it to Tokyo | 22:18 |
rm_work | crc32: switch back to master, do the requirements.txt change, review it, then cherry-pick the latest commit hash from your TLS change, then review again | 22:18 |
rm_work | that should be the easiest way | 22:18 |
rm_work | I have done that probably about 10 times so far <_< | 22:19 |
crc32 | so am I making the change request or not for requirments.txt? | 22:19 |
rm_work | well, it should be separate, and if no one is around and no one cares about ATC stuff, then go ahead and do it yourself | 22:19 |
crc32 | was there really a sumit going on in tokyo or was I just making that up? | 22:20 |
rm_work | don't want things to get TOO complicated :P | 22:20 |
rm_work | crc32: I have heard that the next Fall summit is there | 22:20 |
rm_work | I don't know where that was officially stated though | 22:20 |
crc32 | yea I think I was making it up as a joke. | 22:20 |
a2hill | Its on the sweaters :P | 22:21 |
a2hill | http://openstackdays.com/en/ | 22:21 |
a2hill | sola city | 22:21 |
a2hill | er | 22:22 |
rm_work | I think that was a different thing tho | 22:22 |
a2hill | yea | 22:22 |
crc32 | yet it says 2014 | 22:22 |
a2hill | why is it on the sweater then :P | 22:22 |
rm_work | ok well | 22:23 |
rm_work | https://twitter.com/openstack/status/529554010865680384 | 22:23 |
rm_work | there | 22:23 |
a2hill | awesome! | 22:23 |
crc32 | I don't know. Sounds like something I made up back in the summer. | 22:24 |
rm_work | I first heard it from clev I think | 22:24 |
crc32 | How are people getting the change ID to appear above the "partially-Implemnts: blah blah" line in their commits? | 22:26 |
rm_work | it doesn't really matter | 22:27 |
rm_work | but | 22:27 |
rm_work | it will respect whatever order you put it in | 22:27 |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Do not list SQLAlchemy < 0.9.7 in requirments.txt https://review.openstack.org/143738 | 22:28 |
crc32 | like that? | 22:29 |
rm_work | yep | 22:29 |
rm_work | looks good | 22:29 |
rm_work | now cherry-pick e9c01f4a1d9fec2829381234ef568a7feb46fbcd on top | 22:29 |
rm_work | (your latest TLS change) | 22:30 |
crc32 | danmit. Conflict resolution. Both commits already have the requirments.txt file change. | 22:31 |
rm_work | right | 22:32 |
rm_work | easy to fix | 22:32 |
rm_work | just fix that and do whatever it told you to do | 22:32 |
rm_work | either `git cherry-pick --continue` or `git rebase --continue` or possibly just `git commit` | 22:32 |
rm_work | I always forget which one it asks you to do, but it should tell you | 22:33 |
crc32 | I did git --allow-empty | 22:33 |
crc32 | and poof | 22:33 |
rm_work | uhhh wat | 22:34 |
rm_work | i have never heard of that | 22:34 |
rm_work | what does that even do? | 22:34 |
crc32 | borked everything up apparently. In the graph it tried to squeeze the SQL commit before the TLS commit. | 22:35 |
rm_work | right, that is what you want | 22:36 |
crc32 | ok lets try this again | 22:37 |
crc32 | It shows the TLS commit occuring before the SQLAlchemy fix. Yet the requirments file is updated. Then the SQLAlchemy commit is ontop as an empty commit. | 22:40 |
crc32 | is that expected? | 22:40 |
rm_work | no | 22:41 |
rm_work | do this | 22:41 |
rm_work | git review -d 142915 && git review -d 143738 | 22:42 |
rm_work | that'll ensure you have the two latest branches | 22:42 |
rm_work | and you should end on the requirements change | 22:42 |
rm_work | then do | 22:43 |
rm_work | git cherry-pick e9c01f4a1d9fec2829381234ef568a7feb46fbcd | 22:43 |
rm_work | it should *just work* | 22:43 |
rm_work | it does on mine | 22:43 |
rm_work | then just `git review` | 22:43 |
crc32 | run those commands from master right? | 22:43 |
rm_work | from wherever | 22:43 |
rm_work | it won't matter | 22:43 |
rm_work | it changes your branch around | 22:43 |
crc32 | FatelException in garrit.dll | 22:44 |
rm_work | what | 22:44 |
crc32 | just kidding | 22:44 |
rm_work | lol | 22:44 |
rm_work | it's not that crazy, considering two weeks ago I had issues with OSX complaining about python.exe <_< | 22:45 |
crc32 | ok cool. So its making the SQL commit a dependency. | 22:45 |
rm_work | yes | 22:46 |
crc32 | how do I submit this as a review of https://review.openstack.org/#/c/142915/ | 22:47 |
rm_work | just type "git review" now | 22:48 |
rm_work | it should be good | 22:48 |
rm_work | it'll ask you if you're sure you want to commit | 22:48 |
rm_work | and list two commit hashes | 22:48 |
rm_work | and they should be correct | 22:48 |
rm_work | it tracks based on the "Change-Id" line in the commit, not based on the branch or anything, so it will know where to put it | 22:49 |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 22:49 |
rm_work | perfect | 22:49 |
rm_work | so now we should be able to get https://review.openstack.org/#/c/143738/ approved quickly | 22:49 |
rm_work | and everything will just be fine | 22:49 |
crc32 | Hmm. Wonderr if it will git rejected due to profanity in the commit message. | 22:50 |
rm_work | lol | 22:50 |
rm_work | "bork"? :P | 22:50 |
rm_work | I think that's fine | 22:50 |
*** mlavalle has quit IRC | 22:52 | |
*** crc32 has quit IRC | 22:53 | |
*** mlavalle has joined #openstack-lbaas | 22:54 | |
*** mlavalle has quit IRC | 22:59 | |
*** mlavalle has joined #openstack-lbaas | 22:59 | |
*** sbfox has quit IRC | 23:03 | |
*** hichtakk has quit IRC | 23:12 | |
*** hichtakk has joined #openstack-lbaas | 23:12 | |
*** rm_work is now known as rm_work|away | 23:16 | |
*** hichtakk has quit IRC | 23:47 | |
*** ajmiller has quit IRC | 23:52 | |
*** mlavalle has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!