https://bugzilla.redhat.com/show_bug.cgi?id=1584992
Nigel Babu changed:
What|Removed |Added
CC||nig...@redhat.com
--- Comment #1 from
https://bugzilla.redhat.com/show_bug.cgi?id=1585055
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1584998
--- Comment #1 from Amar Tumballi ---
Notice that this has a problem that we can get into infinite loop of warning
people every 2weeks of inactivity, but never reaching 90days. Hence we need a
mechanism to identify the patches only once, ie,
https://bugzilla.redhat.com/show_bug.cgi?id=1585055
Bug ID: 1585055
Summary: Moving parsefuse to gluster project on Github
Product: GlusterFS
Version: mainline
Component: project-infrastructure
Assignee: b...@gluster.org
https://bugzilla.redhat.com/show_bug.cgi?id=1584998
Bug ID: 1584998
Summary: need automatic inclusion of few reviewers to a given
patch (after 2 weeks)
Product: GlusterFS
Version: mainline
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1584992
Bug ID: 1584992
Summary: Need python pep8 and other relevant tests in smoke if
a patch includes any python file
Product: GlusterFS
Version: mainline
Component:
https://bugzilla.redhat.com/show_bug.cgi?id=1577668
Shyamsundar changed:
What|Removed |Added
Blocks|1575386 (glusterfs-4.1.0) |
Referenced
https://bugzilla.redhat.com/show_bug.cgi?id=1580503
Bug ID: 1580503
Summary: Privacy alias needed
Product: GlusterFS
Version: mainline
Component: website
Assignee: b...@gluster.org
Reporter: a...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1564119
Amar Tumballi changed:
What|Removed |Added
Status|NEW |MODIFIED
---
https://bugzilla.redhat.com/show_bug.cgi?id=1578295
M. Scherer changed:
What|Removed |Added
CC||msche...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1570161
Nigel Babu changed:
What|Removed |Added
CC||nig...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1578295
Bug ID: 1578295
Summary: Experiment with havaged on gerrit server
Product: GlusterFS
Version: mainline
Component: project-infrastructure
Assignee: b...@gluster.org
https://bugzilla.redhat.com/show_bug.cgi?id=1577668
Nigel Babu changed:
What|Removed |Added
Status|NEW |POST
--- Comment #2
https://bugzilla.redhat.com/show_bug.cgi?id=1577668
sankarshan changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1577668
Bug ID: 1577668
Summary: Regressions/smoke/other tests failing in build (due to
new deprecated warnings)
Product: GlusterFS
Version: mainline
Component:
of Bugzilla check in smoke.
1. No Reference (BUG/Issue) ID
2. Product is not GlusterFS
3. Posted Bug Status is not valid
4. Posted Bug is not marked for relevant release version. (like a bug of 4.0
version is posted for release-4.1 branch).
Would be great to see the reason coming out to gerrit
https://bugzilla.redhat.com/show_bug.cgi?id=1575487
Bug ID: 1575487
Summary: New machine for workerant
Product: GlusterFS
Version: mainline
Component: project-infrastructure
Assignee: b...@gluster.org
Reporter:
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
Nigel Babu changed:
What|Removed |Added
Status|MODIFIED|CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1556834
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1575388
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1575387
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1556834
--- Comment #1 from Shyamsundar ---
Ping...
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=HloGGo6JhP=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=1575388
Bug ID: 1575388
Summary: Request gerrit dashboard addition for 4.1
Product: GlusterFS
Version: 4.1
Component: project-infrastructure
Assignee: b...@gluster.org
Reporter:
https://bugzilla.redhat.com/show_bug.cgi?id=1575387
Bug ID: 1575387
Summary: Restrict gerrit merge permissions for branch
release-4.1 to release owners
Product: GlusterFS
Version: 4.1
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
--- Comment #7 from M. Scherer ---
I have added the access 2 days ago, seems I forgot to announce on this bug
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/show_bug.cgi?id=1574984
Bug ID: 1574984
Summary: Switch 5 builders in the cage to Fedora 28
Product: GlusterFS
Version: mainline
Component: project-infrastructure
Assignee: b...@gluster.org
https://bugzilla.redhat.com/show_bug.cgi?id=1573488
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
Deepshikha khandelwal changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1573783
Bug ID: 1573783
Summary: Few jenkins plugins need to be installed
Product: GlusterFS
Version: 3.13
Component: project-infrastructure
Assignee: b...@gluster.org
Reporter:
https://bugzilla.redhat.com/show_bug.cgi?id=1573488
Bug ID: 1573488
Summary: infra: comment-on-issue can't handle anything except
ASCII
Product: GlusterFS
Version: mainline
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #30 from Xavi Hernandez ---
(In reply to Jeff Darcy from comment #24)
> Personally I'm willing to go with the flow on those. I've laid out my
> reasoning, other people seem less inclined to compromise, I think
https://bugzilla.redhat.com/show_bug.cgi?id=1570161
Amye Scavarda changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1570161
--- Comment #2 from Amye Scavarda ---
Ping on this?
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=agOoRGLgjj=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #29 from Nigel Babu ---
Um, so I've been accepting the pull requests that come in more or less mostly
so we can see what the output looks like. I'm going to wait for 24h for each of
the remaining pull requests
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #28 from Xavi Hernandez ---
(In reply to Aravinda VK from comment #27)
> I prefer Right(`AlignEscapedNewlines=Right`), since Patch will have minimum
> number of lines change on modification. For example,
>
>
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #27 from Aravinda VK ---
(In reply to Xavi Hernandez from comment #19)
> I made a mistake on one of the proposed values:
>
> AlignEscapedNewLinesLeft=false
>
> What I wanted to propose was to set it to
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #25 from Xavi Hernandez ---
I don't want to start any battle. It was requested to create pull requests with
the proposed changes and that's what I've done. I've not hidden that some of
the changes are
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #23 from Xavi Hernandez ---
The change will also include some of the controversial options:
IndentCaseLabels=true
SpaceBeforeParens=ControlStatements
What should we do with them ?
--
You are receiving this
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #22 from Xavi Hernandez ---
Some issues I've seen:
* Some initializations are formatted like this:
char gfid_local[GF_UUID_BUF_SIZE] = {
0,
};
I've tried to change
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #21 from Jeff Darcy ---
The feedback so far has been strongly positive, but one issue has been raised.
As Xavi mentioned, the PointerAlignment doesn't seem to work properly on lines
affected by AlignConsecutive*.
https://bugzilla.redhat.com/show_bug.cgi?id=1570161
--- Comment #1 from Amye Scavarda ---
Editing to add:
Archives are private, not that the list should be invisible.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/show_bug.cgi?id=1563379
Nigel Babu changed:
What|Removed |Added
Flags|needinfo?(nig...@redhat.com |
|)
https://bugzilla.redhat.com/show_bug.cgi?id=1570576
Nigel Babu changed:
What|Removed |Added
Summary|Common on issue should |Comment on issue should
https://bugzilla.redhat.com/show_bug.cgi?id=1570576
Bug ID: 1570576
Summary: Common on issue should leave a comment with the exact
issue on the review
Product: GlusterFS
Version: mainline
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #19 from Xavi Hernandez ---
I made a mistake on one of the proposed values:
AlignEscapedNewLinesLeft=false
What I wanted to propose was to set it to 'true' (which matches the explanation
I gave).
--
You
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #18 from Jeff Darcy ---
Monday morning EST would be fine.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #17 from Amar Tumballi ---
Thanks for these pointers Jeff, makes the case even more critical to work
towards.
Shyam kindly agreed to make this a blocker for 4.1 branching out, which means,
we have to come to
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #16 from Jeff Darcy ---
FWIW, since this came up in the maintainers' meeting...
The inconsistency of style has had a measurable negative impact on ramp-up time
for new developers at Facebook. I'm sure the same
https://bugzilla.redhat.com/show_bug.cgi?id=1568488
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #15 from Xavi Hernandez ---
(In reply to Jeff Darcy from comment #13)
> IndentCaseLabels: personally I don't like indenting case labels because they
> create a gap with no close brace - unless people put
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
Amar Tumballi changed:
What|Removed |Added
Priority|unspecified |high
---
https://bugzilla.redhat.com/show_bug.cgi?id=1568488
Nigel Babu changed:
What|Removed |Added
CC||nig...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #13 from Jeff Darcy ---
IndentCaseLabels: personally I don't like indenting case labels because they
create a gap with no close brace - unless people put braces around cases, which
is a bad habit for other reasons
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #12 from Xavi Hernandez ---
Using Chromium as base, basically the changes you propose seem ok. I would like
to propose these additional changes:
* I prefer to use 4 spaces for indent. 2 seems too few to me.
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #11 from Jeff Darcy ---
Whatever we pick is going to require some tweaking. Some of the things I
notice:
* All use two-space indents, which I personally find atrocious.
* All put a function's type on the same
https://bugzilla.redhat.com/show_bug.cgi?id=1568488
Bug ID: 1568488
Summary: Is fstat.gluster.org capturing all the regression
failures?
Product: GlusterFS
Version: mainline
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1549000
Amar Tumballi changed:
What|Removed |Added
Status|MODIFIED|CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #10 from Amar Tumballi ---
(Update from Nigel)
Which format should we start with as base? Google/LLVM/Mozilla/Webkit/Chromium
*
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #9 from Amar Tumballi ---
(In reply to Nigel Babu from comment #7)
> Alright. So we have agreement that this is a good idea. How do we want to
> implement this? As a check or as a pre-commit hook?
This is a
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #8 from Amar Tumballi ---
(In reply to Shreyas Siravara from comment #6)
> Whoever came up with this idea is a genius, and I totally ack it :)
All credits for this to Shreyas Siravara! As the original idea
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
Nigel Babu changed:
What|Removed |Added
CC||nig...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1564115
Nigel Babu changed:
What|Removed |Added
Status|NEW |POST
--- Comment #5
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
--- Comment #5 from M. Scherer ---
I am also ok with all access, if someone explain the best practice regarding
ssh security and stuff like this. (ideally, key on a smartcard/yubikey, but at
minima, password protected ssh
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
Worker Ant <bugzilla-...@gluster.org> changed:
What|Removed |Added
Status|POST|MO
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
--- Comment #3 from Nigel Babu ---
Yeah, we have the option of either setting two levels or giving all access and
depending on trust.
Right now, I'm looking for all Jenkins nodes, build.gluster.org, and the
associated
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
M. Scherer changed:
What|Removed |Added
CC||msche...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1567880
Bug ID: 1567880
Summary: Grant Deepshikha access to all CI-related
infrastructure
Product: GlusterFS
Version: mainline
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1549662
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1567221
Bug ID: 1567221
Summary: Issue checking should ensure required flags are
present, even on newer patchsets in gerrit
Product: GlusterFS
Version: mainline
Component:
https://bugzilla.redhat.com/show_bug.cgi?id=1491156
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1493408
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1558335
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1545003
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1377533
Amar Tumballi changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1382097
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1563379
Kaleb KEITHLEY changed:
What|Removed |Added
Flags|
https://bugzilla.redhat.com/show_bug.cgi?id=1551513
Ramakrishna Reddy Yekulla changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1566449
Bug ID: 1566449
Summary: Disable the + ban for email on the mailing list
Product: GlusterFS
Version: mainline
Component: project-infrastructure
Assignee: b...@gluster.org
https://bugzilla.redhat.com/show_bug.cgi?id=1564139
Nigel Babu changed:
What|Removed |Added
Resolution|NOTABUG |CURRENTRELEASE
--
https://bugzilla.redhat.com/show_bug.cgi?id=1564139
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1564139
--- Comment #3 from Nigel Babu ---
This has now landed. Notable missing feature - if there is an existing issue on
the commit, we don't check if it has all the flags. Going to keep this issue
open till I fix that.
--
You
https://bugzilla.redhat.com/show_bug.cgi?id=1562699
Nigel Babu changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1562132
Nigel Babu changed:
What|Removed |Added
Status|NEW |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1564372
--- Comment #2 from Nigel Babu ---
I'd say alert to a list like ale...@gluster.org. We'll still do best effort
working day coverage. This only enhances our ability to see what fails sooner
than someone else noticing the
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
Shreyas Siravara changed:
What|Removed |Added
CC||sshre...@fb.com
https://bugzilla.redhat.com/show_bug.cgi?id=1564451
Bug ID: 1564451
Summary: The abandon job for patches should post info in
bugzilla that some patch is abandon'd.
Product: GlusterFS
Version: mainline
Component: project
https://bugzilla.redhat.com/show_bug.cgi?id=1564139
--- Comment #2 from Amar Tumballi ---
The note to community was that we are willing to make edits to behavior on
discussion, and will go ahead and implement soon for making sure we have
documentation for the features. Can
https://bugzilla.redhat.com/show_bug.cgi?id=1564372
Bug ID: 1564372
Summary: Setup Nagios server
Product: GlusterFS
Version: 4.0
Component: project-infrastructure
Assignee: b...@gluster.org
Reporter: nig...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1564115
--- Comment #4 from Amar Tumballi ---
No serious need of 'Vote' at present. This can be 'SUCCESS', 'FAILURE', ABORT
or whatever.. as the person triggering the run, I would look up the result
before voting.
Lets get the
at.com
--- Comment #1 from Nigel Babu <nig...@redhat.com> ---
1. Referencing the wrong issue is not something we can control. You can always
make a mistake in either smoke or bugzilla. That's something I cannot control.
2. The code for ensuring the rest is ready for merge pending agreement from t
https://bugzilla.redhat.com/show_bug.cgi?id=1564115
Nigel Babu changed:
What|Removed |Added
CC||nig...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #5 from Jeff Darcy ---
I think we're all in agreement on spaces vs. tabs. Tabs are eight spaces wide,
full stop, end of story. It's kind of survivable with eight-space indents, but
any other indent width becomes a
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #4 from Shyamsundar ---
Overall ack to the proposal.
Nice call-out by Jeff on any other change requiring a big-bang addressing to be
a part of the same.
I can live with 4 spaces :), but no tabs (way too much
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
Xavi Hernandez changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1564149
Bug ID: 1564149
Summary: Agree upon a coding standard, and automate check for
this in smoke
Product: GlusterFS
Version: mainline
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1563379
Kaleb KEITHLEY changed:
What|Removed |Added
Keywords||Triaged
https://bugzilla.redhat.com/show_bug.cgi?id=1564139
Bug ID: 1564139
Summary: need a test to validate flags and validity of github
issues.
Product: GlusterFS
Version: mainline
Component: project-infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1563379
Nigel Babu changed:
What|Removed |Added
CC||nig...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1564115
Bug ID: 1564115
Summary: need option 'run brick-mux-tests' in reviews
Product: GlusterFS
Version: mainline
Component: project-infrastructure
Severity: high
Assignee:
https://bugzilla.redhat.com/show_bug.cgi?id=1563279
Kaleb KEITHLEY changed:
What|Removed |Added
Status|NEW |CLOSED
1201 - 1300 of 2537 matches
Mail list logo