Re: [Gluster-devel] FreeBSD smoke test may fail for older changes, rebase needed

2018-07-28 Thread Niels de Vos
On Sat, Jul 28, 2018 at 03:37:46PM +0200, Niels de Vos wrote:
> This Friday argp-standalone got installed on the FreeBSD Jenkins
> slave(s). With the library available, we can now drop the bundled and
> unmaintained contrib/argp-standlone/ from our glusterfs sources.
> 
> Unfortunately building on FreeBSD fails if the header/library is
> installed. This has been corrected with https://review.gluster.org/20581
> but that means changes posted in Gerrit may need a rebase to include the
> fix for building on FreeBSD.
> 
> I think I have rebased all related changes that did not have negative
> comments asking for corrections/improvement. In case I missed a change,
> please rebase your patch so the smoke test runs again.
> 
> Sorry for any inconvenience that this caused,
> Niels

It just occured to me that the argp-standalone installation also affects
the release-4.1 and release-3.12 branches. Jiffin, Shyam, do you want to
cherry-pick https://review.gluster.org/20581 to fix that, or do you
prefer an alternative that always uses the bundled version of the
library?

Niels


signature.asc
Description: PGP signature
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel

Re: [Gluster-devel] FreeBSD smoke test may fail for older changes, rebase needed

2018-07-28 Thread Amar Tumballi
On Sat, Jul 28, 2018 at 7:07 PM, Niels de Vos  wrote:

> This Friday argp-standalone got installed on the FreeBSD Jenkins
> slave(s). With the library available, we can now drop the bundled and
> unmaintained contrib/argp-standlone/ from our glusterfs sources.
>
> Unfortunately building on FreeBSD fails if the header/library is
> installed. This has been corrected with https://review.gluster.org/20581
> but that means changes posted in Gerrit may need a rebase to include the
> fix for building on FreeBSD.
>
> I think I have rebased all related changes that did not have negative
> comments asking for corrections/improvement. In case I missed a change,
> please rebase your patch so the smoke test runs again.
>
>
Thanks for the changes Niels, this makes project maintained easier with
available libraries instead of carrying that code.


> Sorry for any inconvenience that this caused,
>

It happens with some changes like this. Thanks for updating everyone about
this, and also triggering the build again after rebasing.

-Amar

Niels
>
> ___
> Gluster-devel mailing list
> Gluster-devel@gluster.org
> https://lists.gluster.org/mailman/listinfo/gluster-devel
>



-- 
Amar Tumballi (amarts)
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel

[Gluster-devel] FreeBSD smoke test may fail for older changes, rebase needed

2018-07-28 Thread Niels de Vos
This Friday argp-standalone got installed on the FreeBSD Jenkins
slave(s). With the library available, we can now drop the bundled and
unmaintained contrib/argp-standlone/ from our glusterfs sources.

Unfortunately building on FreeBSD fails if the header/library is
installed. This has been corrected with https://review.gluster.org/20581
but that means changes posted in Gerrit may need a rebase to include the
fix for building on FreeBSD.

I think I have rebased all related changes that did not have negative
comments asking for corrections/improvement. In case I missed a change,
please rebase your patch so the smoke test runs again.

Sorry for any inconvenience that this caused,
Niels


signature.asc
Description: PGP signature
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel

Re: [Gluster-devel] Gerrit downtime on Aug 8, 2016

2018-07-28 Thread Nigel Babu
FYI: There is an issue with seeing diffs on staging. I've root caused this
to a bug in our apache configuration for Gerrit. This is more tricky than I
want to handle at the moment, but I'm aware of the problem and tested out a
fix. We'll fix it more permanently in ansible on Monday. My fix will get
overwritten by Ansible tonight :)

On Fri, Jul 27, 2018 at 5:28 PM Nigel Babu  wrote:

> Hello,
>
> It's been a while since we upgraded Gerrit. We plan to do a full upgrade
> and move to 2.15.3. Among other changes, this brings in the new PolyGerrit
> interface which brings significant frontend changes. You can take a look at
> how this would look on the staging site[1].
>
> ## Outage Window
> 0330 EDT to 0730 EDT
> 0730 UTC to 1130 UTC
> 1300 IST to 1700 IST
>
> The actual time needed for the upgrade is about than hour, but we want to
> keep a larger window open to rollback in the event of any problems during
> the upgrade.
>
> --
> nigelb
>


-- 
nigelb
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel

Re: [Gluster-devel] ./tests/00-geo-rep/georep-basic-dr-rsync.t fails

2018-07-28 Thread Raghavendra Gowdappa
Few failures on https://review.gluster.org/#/c/20576/ seen too.

On Sat, Jul 28, 2018 at 2:47 PM, Raghavendra Gowdappa 
wrote:

> Kotresh,
>
> The test failed on master (without the patch) too. I've seen failures on
> this earlier too.
>

https://build.gluster.org/job/centos7-regression/1934/


> regards,
> Raghavendra
>
>
>
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel

[Gluster-devel] ./tests/00-geo-rep/georep-basic-dr-rsync.t fails

2018-07-28 Thread Raghavendra Gowdappa
Kotresh,

The test failed on master (without the patch) too. I've seen failures on
this earlier too.

regards,
Raghavendra
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel