Time |
Nick |
Message |
01:08 |
|
an joined #gluster-dev |
01:12 |
|
bala joined #gluster-dev |
01:40 |
|
aviksil_ joined #gluster-dev |
02:28 |
|
bharata-rao joined #gluster-dev |
03:13 |
|
anoopcs joined #gluster-dev |
03:17 |
|
aravindavk joined #gluster-dev |
03:26 |
|
shubhendu joined #gluster-dev |
03:35 |
|
nishanth joined #gluster-dev |
03:50 |
|
itisravi joined #gluster-dev |
03:52 |
|
kshlm joined #gluster-dev |
04:30 |
|
anoopcs joined #gluster-dev |
04:32 |
|
Rafi_kc joined #gluster-dev |
04:32 |
|
rafi1 joined #gluster-dev |
05:02 |
|
jiffin joined #gluster-dev |
05:04 |
|
kdhananjay joined #gluster-dev |
05:11 |
|
spandit joined #gluster-dev |
05:12 |
|
ndarshan joined #gluster-dev |
05:15 |
|
atalur joined #gluster-dev |
05:16 |
|
kdhananjay joined #gluster-dev |
05:18 |
|
deepakcs joined #gluster-dev |
05:24 |
|
Gaurav__ joined #gluster-dev |
05:29 |
|
aviksil_ joined #gluster-dev |
05:31 |
|
atinmu joined #gluster-dev |
05:40 |
|
raghu joined #gluster-dev |
05:42 |
|
ppai joined #gluster-dev |
06:02 |
|
bala joined #gluster-dev |
06:08 |
|
lalatenduM joined #gluster-dev |
06:11 |
|
hagarth joined #gluster-dev |
06:21 |
|
_Bryan_ joined #gluster-dev |
06:31 |
|
purpleidea joined #gluster-dev |
06:55 |
|
bala joined #gluster-dev |
07:28 |
|
deepakcs joined #gluster-dev |
07:32 |
ndevos |
wow, hchiramm, you're fast@ |
07:32 |
ndevos |
! |
07:32 |
Humble |
ndevos, :P :) |
07:33 |
Humble |
it depends on the window which is active on your browser :) |
07:33 |
ndevos |
Humble: I think we broke Gerrit on that commit, the change does not have our names as reviewer... oh well |
07:34 |
hagarth |
do we have a problem with Gerrit now? |
07:34 |
* Humble |
not seeing any issues |
07:34 |
ndevos |
hmm, maybe? the patch I just wanted to merge is in status: Submitted, Merge Pending |
07:34 |
ndevos |
Submitted, Merge Pending |
07:35 |
ndevos |
copy/paste fail -> http://review.gluster.org/8855 |
07:36 |
Humble |
but I see only one patch is in that status |
07:36 |
Humble |
is// |
07:37 |
ndevos |
and patch #6 has the updated commit message with my review: http://review.gluster.org/#/c/8855/6//COMMIT_MSG |
07:37 |
hagarth |
ndevos: submitted, merge pending is a bad state :( |
07:38 |
hagarth |
ndevos: that is symptomatic of the problem I had with release-3.6 a few weeks back |
07:38 |
hagarth |
we need to review carefully if there are missing patches.. |
07:39 |
ndevos |
hagarth: I'm not sure what you did, but Gerrit still shows the same status for that patch? |
07:39 |
hagarth |
ndevos: it is best to abandon such patches and send out new ones |
07:39 |
ndevos |
argh... |
07:41 |
Humble |
yep , there had some issues in past and atleast 3 were in merge pending status.. then realized some of the merges were not recorded. |
07:43 |
Humble |
ndevos, do u want to review http://review.gluster.org/#/c/8866/ again / |
07:44 |
ndevos |
Humble: nah, thats not needed |
07:44 |
Humble |
ok.. :) |
07:45 |
ndevos |
Humble, hagarth: uh, sending a new patch gets into the same state... http://review.gluster.org/8866 |
07:47 |
* ndevos |
has no idea what he can do about it, and will make himself a coffee |
07:47 |
ndevos |
coffee++ |
07:47 |
glusterbot |
ndevos: coffee's karma is now 1 |
07:58 |
hagarth |
ndevos: 8866 is merged? |
07:58 |
hagarth |
coffee++ too |
07:58 |
glusterbot |
hagarth: coffee's karma is now 2 |
08:00 |
|
vimal joined #gluster-dev |
08:00 |
ndevos |
hagarth: no, both are in "Submitted, merged pending" status |
08:01 |
ndevos |
last commit in master, after fetching is http://review.gluster.org/8761 |
08:01 |
ndevos |
which matches http://review.gluster.org/#/q/project:glusterfs+status:merged,n,z |
08:01 |
hagarth |
right .. saw that now |
08:02 |
hagarth |
let me check if I can notice anything in r.g.o |
08:02 |
ndevos |
only these two patches seem to be in this state: http://review.gluster.org/#/q/project:glusterfs+status:submitted,n,z |
08:02 |
ndevos |
okay, let me know if you want me to do anything |
08:03 |
hagarth |
there are primary key violations as per gerrit logs ;) |
08:07 |
|
xavih joined #gluster-dev |
08:07 |
ndevos |
*gulp* |
08:42 |
|
ppai joined #gluster-dev |
08:43 |
hagarth |
ndevos: what is the commit id of the patch you submitted in your local repository? |
08:44 |
ndevos |
hagarth: fc86e9ae2aa7719b01ec9b45aab2f04e8372df02 |
08:44 |
ndevos |
hagarth: that would be the 2nd patch, http://review.gluster.org/8866 |
08:44 |
hagarth |
ndevos: right |
08:46 |
ndevos |
hagarth: Gerrit munged the 1st patch and added the review url in the commit message, that resulted in commit 8d2dd5976c33b3f261051e43e6d9b0cb8ad37ccf |
08:46 |
ndevos |
http://review.gluster.org/8855 patch #6 |
08:47 |
hagarth |
ndevos: abandoned it now |
08:48 |
ndevos |
hagarth: okay, thanks, you must have super powers :) |
08:48 |
ndevos |
hagarth: so, maybe abandon 8866 too? |
08:48 |
hagarth |
ndevos: can you re-submit the patch by ammending the change-id? |
08:48 |
* ndevos |
has an abandon button on that one |
08:48 |
hagarth |
ndevos: done that as well :D |
08:48 |
ndevos |
oh, lol |
08:49 |
ndevos |
http://review.gluster.org/8867 is up |
08:50 |
ndevos |
hagarth: you want to try and submit that one? |
08:50 |
hagarth |
ndevos: yeah, maybe let me do that |
08:51 |
hagarth |
ndevos: done! |
08:52 |
ndevos |
hagarth++ thanks, you fixed it! |
08:52 |
glusterbot |
ndevos: hagarth's karma is now 12 |
08:52 |
hagarth |
probably abandon + re-submit with a different change-id is the workaround we can use for these scenarios |
08:53 |
hagarth |
several folks have suggested tweaking db through the gsql interface .. but probably not worth it if this workaround continues to work |
08:53 |
ndevos |
okay, and only admins and original submitters can abandon patches |
08:53 |
hagarth |
ndevos: right |
08:53 |
ndevos |
who are the Gerrit admins that can abandon any patch, in case the original submitter is not available? |
08:54 |
hagarth |
ndevos: me, avati and Humble atm |
08:54 |
ndevos |
okay, that should cover my needs :) |
08:55 |
Humble |
ndevos, gluster infra is notified about the issue.. |
08:55 |
hagarth |
i hope we don't run into this again |
08:55 |
hagarth |
however it has happened a few times in the recent past |
08:55 |
Humble |
please feel free to add details, if any. |
08:56 |
ndevos |
and this is supposed to be fixed with a newer Gerrit version? |
08:56 |
hagarth |
ndevos: not quite :( |
08:56 |
hagarth |
folks had reported seeing this in 2.8.4 too |
08:56 |
ndevos |
oh :-/ |
08:56 |
hagarth |
not sure about 2.9 |
08:59 |
Humble |
hagarth, is it a known issue ? I remember u had references , but I am not sure this was the issue covered there |
09:00 |
hagarth |
Humble: yes, a known issue with gerrit |
09:00 |
Humble |
oh..ok |
09:01 |
hagarth |
I had posted some links in the release-3.6 patches which describe the issue better |
09:01 |
Humble |
oh.. ok.. |
09:04 |
|
kanagaraj joined #gluster-dev |
09:10 |
xavih |
I've a bug that was only partially solved. Do I put it again in the ASSIGNED state and send another patch or I have to create a new bug id ? |
09:24 |
|
hagarth joined #gluster-dev |
09:29 |
|
RaSTar joined #gluster-dev |
09:32 |
ndevos |
xavih: whatever you like :) |
09:34 |
xavih |
ndevos: thanks. I'll reuse the same bug... :) |
09:35 |
ndevos |
xavih: a new bug is easier to track... you can then also close the old bug as a duplicate of the new one |
09:35 |
|
spandit joined #gluster-dev |
09:36 |
* ndevos |
has to think about how to detect FailedQA bugs and not move them to ON_QA with a next release if there wasnt an other patch |
09:37 |
xavih |
ndevos: the bug solved some issues between 32/64 bits architectures, but another patch sent before but not yet merged also had some issues with 32/64 bits that the patch didn't solve |
09:38 |
xavih |
ndevos: I need to send a patch for this newly added issues |
09:38 |
ndevos |
xavih: okay, so it is something like a follow-up patch? |
09:39 |
ndevos |
xavih: anyway, a new bug is easier to track, you can add a reference in the 1st bug to the 2nd and explain why the 2nd bug+patch fix a similar issue |
09:39 |
ndevos |
xavih: but well, feel free to do whatever you think makes most sense :) |
09:40 |
xavih |
ndevos: yes, the first patch was incomplete because it only solved the issues currently present on master |
09:40 |
xavih |
ndevos: ok, then I'll create a new one |
09:40 |
ndevos |
xavih: right, so the 1st bug is solved, and a 2nd bug got introduces somewhere along the way |
09:40 |
ndevos |
*introduced |
09:40 |
xavih |
ndevos: yes, it can be seen this way :P |
09:41 |
ndevos |
xavih: I see it that way, and it makes things easier ;-) |
09:41 |
xavih |
ndevos: then I'll create another bug for this new problem |
09:41 |
xavih |
:) |
09:41 |
xavih |
ndevos: thanks :) |
09:42 |
ndevos |
xavih: sounds good to me, thanks1 |
09:42 |
ndevos |
uh, ! |
09:52 |
|
bala joined #gluster-dev |
10:00 |
|
kanagaraj joined #gluster-dev |
10:00 |
|
kshlm joined #gluster-dev |
10:17 |
|
hagarth joined #gluster-dev |
10:26 |
|
an joined #gluster-dev |
10:44 |
|
ira joined #gluster-dev |
10:50 |
|
shyam joined #gluster-dev |
11:33 |
|
tdasilva joined #gluster-dev |
11:35 |
|
shyam joined #gluster-dev |
11:43 |
|
kkeithley joined #gluster-dev |
11:49 |
kkeithley |
ndevos: phone call! |
11:51 |
|
ppai joined #gluster-dev |
12:16 |
|
itisravi_ joined #gluster-dev |
12:30 |
|
itisravi joined #gluster-dev |
13:36 |
|
aviksil_ joined #gluster-dev |
13:40 |
|
shyam joined #gluster-dev |
13:55 |
|
anoopcs joined #gluster-dev |
14:36 |
|
jobewan joined #gluster-dev |
14:38 |
|
lalatenduM joined #gluster-dev |
15:46 |
|
hagarth joined #gluster-dev |
16:01 |
JustinClift |
ndevos: Ahhh, good workaround there for the Gerrit problem. |
16:02 |
* JustinClift |
wasn't sure how we're handling that |
16:03 |
hagarth |
JustinClift: I think we should take up gerrit upgrade sometime soon .. once you are done with the backups etc. |
16:03 |
hagarth |
I would also like to integrate sonar with gerrit and the current lower version of gerrit hampers us |
16:06 |
JustinClift |
hagarth: Makes sense |
16:06 |
JustinClift |
I've kind of been letting everything else get in front of the backups stuff |
16:06 |
JustinClift |
I need to figure out how to backup the H2 database in gerrit properly |
16:06 |
JustinClift |
There are several pages open with various methods |
16:06 |
* JustinClift |
goes off to try them now |
16:08 |
|
jiffin joined #gluster-dev |
16:08 |
hagarth |
JustinClift: maybe we can check with the ovirt folks, have seen them doing gerrit/jenkins upgrades |
16:08 |
hagarth |
to see if they have a documented procedure that works |
16:09 |
JustinClift |
hagarth: They have their process online (no documented restore proces though!) |
16:09 |
JustinClift |
hagarth: They use PostgreSQL instead of H2 as the database |
16:09 |
JustinClift |
So, it's not useful to us |
16:10 |
hagarth |
JustinClift: I see, ok |
16:12 |
JustinClift |
hagarth: We may need to adopt an approach of shutting down Gerrit each night, backing up the files, and starting it up again |
16:13 |
JustinClift |
hagarth: Coping the actual files takes less than 30 seconds |
16:13 |
JustinClift |
So, I don't think it would have much impact |
16:13 |
hagarth |
JustinClift: yeah. If it is a short window, we can do that. |
17:17 |
|
vikumar joined #gluster-dev |
17:24 |
|
hagarth joined #gluster-dev |
17:35 |
|
tdasilva joined #gluster-dev |
18:06 |
JustinClift |
sync |
18:12 |
JustinClift |
misc: Do you have time to update the bash package on download.gluster.org again? |
18:12 |
JustinClift |
misc: It's not finding the updated-update either |
18:13 |
JustinClift |
The other boxes all seem fine. Doing the slaves now. |
18:19 |
JustinClift |
misc: Ignore that, I'm on it |
19:06 |
|
an joined #gluster-dev |
20:31 |
lalatenduM |
kkeithley, abt building http://download.gluster.org/pub/gluster/glusterfs/nfs-ganesha/RHEL/epel-7/SRPMS/ without building fsal for ceph |
20:32 |
lalatenduM |
kkeithley, I have just have to change "global with_cephfs false" right? |
20:32 |
lalatenduM |
kkeithley, I mean "global with_cephfs true" to "global with_cephfs false" |
20:50 |
JustinClift |
Hmmm, I have a test restored Gerrit server working in a VM |
20:50 |
JustinClift |
OpenID auth isn't working to sign into it though |
20:51 |
* JustinClift |
suspects OpenID server is seeing the VM as some kinda bogus thing |
20:55 |
JustinClift |
Hmmm, bugzilla hooks. Better disable those while mucking around too |
20:57 |
JustinClift |
Ahhh. Isolated VM network, thus the OpenID problem. |
20:59 |
JustinClift |
NOTE - I've just turned off the gerrit server on review.gluster.org (on purpose) |
20:59 |
JustinClift |
Doing a full file level backup of it (to be safe) :) |
20:59 |
JustinClift |
Gimme 5 mins |
21:07 |
JustinClift |
Ok, make that 10 |
21:07 |
JustinClift |
Fixing a few ownership/perms problems on the review box, now that there's a backup |
21:15 |
JustinClift |
k, it's back up |
21:29 |
|
an joined #gluster-dev |
21:32 |
|
MacWinner joined #gluster-dev |
23:09 |
|
MacWinner joined #gluster-dev |