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