Current master branch fails to build

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 28, 2017 at 08:18:50PM +0000, jenkins@xxxxxxxxxxxxxxxxx wrote:
> See <https://build.gluster.org/job/regression-test-with-multiplex/294/display/redirect?page=changes>
> 
> Changes:
> 
> [Jeff Darcy] statedump: add support for dumping basic mem-pool info
> 
> [Jeff Darcy] mem-pool: add tracking of mem_pool that requested the allocation
> 
> [Jeff Darcy] cluster/ec: coverity, fix for BAD_SHIFT
> 
> ------------------------------------------
> Started by timer

... snip!

>   CC     libglusterfs_la-stack.lo
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>: In function ‘gf_proc_dump_mempool_info’:
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>:401: error: ‘glusterfs_ctx_t’ has no member named ‘mempool_list’
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>:401: error: ‘struct mem_pool’ has no member named ‘owner’
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>:401: error: ‘struct mem_pool’ has no member named ‘owner’
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>:401: error: ‘glusterfs_ctx_t’ has no member named ‘mempool_list’
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>:401: error: ‘struct mem_pool’ has no member named ‘owner’
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>:401: error: ‘struct mem_pool’ has no member named ‘owner’
> <https://build.gluster.org/job/regression-test-with-multiplex/ws/libglusterfs/src/statedump.c>:402: error: ‘struct mem_pool’ has no member named ‘active’
> make[3]: *** [libglusterfs_la-statedump.lo] Error 1

This is happening because the mem-pool changes that got merged depend on
https://review.gluster.org/18075 . A revised version to address review
comments is currently undergoing regression testing.

Please review and merge these two patches in the following order:

1. mem-pool: track glusterfs_ctx_t in struct mem_pool
   https://review.gluster.org/18075

2. mem-pool: count allocations done per user-pool
   https://review.gluster.org/18074

In case (1) needs more changes, feel free to modify the change. It is
the easiest to use git-review for that:

   $ git review -r origin -d 18075
   [ edit files ]
   $ git commit -a --fixup=HEAD
   [ test build ]
   $ git rebase -i --autosquash HEAD~2
   $ git review -r origin -R -t rfc master

Thanks!
Niels

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux