Release of Prometheus Ganeti Exporter 1.0.0

Hello all, we are happy to announce the release of version 1.0.0 of the Prometheus Ganeti Exporter: https://github.com/ganeti/prometheus-ganeti-exporter/releases/tag/v1.0.0 Please give it a try and leave feedback, open issues etc. Cheers, Rudi -- Rudolph Bott - b...@sipgate.de Telefon: +49

Welcome to the family: Prometheus Ganeti Exporter

Hey Everyone, Today we are happy to announce a new repository on the Ganeti Github organisation: https://github.com/ganeti/prometheus-ganeti-exporter The code has been kindly donated by the Wikimedia Foundation - thanks a lot! There has been no official release yet. Please test, report any

Re: new HV parameters cfg up-/downgrade?

d one)? > > Thanks, Sascha. > > -- > You received this message because you are subscribed to the Google Groups > "ganeti-devel" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to ganeti-devel+unsubscr...@googlegroups.com.

Re: GHC versions for 3.0?

Hi, i just ask myself, if we try to cover GHC 8.8 do we still have an upgrade path and can build ganeti 3.0 for ubuntu 18.04? Thanks, Ansgar Am Mi., 8. Juli 2020 um 10:19 Uhr schrieb Iustin Pop : > > On 2020-07-08 08:22:22, Rudolph Bott wrote: > > Hi Iustin, > > > >

An update on Ganeti, Google , and the community

Google originally created Ganeti in 2007, for its internal infrastructure, and released it as open source. Over time a healthy community of developers gathered around it, and started contributing, and then fully maintaining it. Since Google's priorities have shifted, we are happy to fully release

[GitHub] ganeti has approved a third-party application that you use

An organization owner has approved the following application to access private data in the ganeti organization: "Travis CI for Open Source" from Travis CI Because you are a member of the ganeti organization and have already granted this application access to your personal accou

Re: Status of migrated issues?

s. > > > On Mon, 29 Jan 2018 at 10:24, Iustin Pop <ius...@google.com> wrote: > >> On 29 January 2018 at 11:08, 'Federico Pareschi' via ganeti-devel < >> ganeti-devel@googlegroups.com> wrote: >> >>> Sorry, this email had completely flown under my radar. >

Re: Status of migrated issues?

January 2018 at 11:08, 'Federico Pareschi' via ganeti-devel < > ganeti-devel@googlegroups.com> wrote: > >> Sorry, this email had completely flown under my radar. >> > > No worries. > > The issues have all been migrated to the github repository, however th

Re: Status of migrated issues?

On 29 January 2018 at 11:08, 'Federico Pareschi' via ganeti-devel < ganeti-devel@googlegroups.com> wrote: > Sorry, this email had completely flown under my radar. > No worries. The issues have all been migrated to the github repository, however there > is a difference

Re: Status of migrated issues?

Sorry, this email had completely flown under my radar. The issues have all been migrated to the github repository, however there is a difference between 'fixed' and 'released' in the old issue tracker on code.google.com. By importing these issues in github, these states turned into tags. I assume

Re: [PATCH master 00/28] Implement the MacVTap device driver feature

t;db...@skroutz.gr> [2016-02-12 15:10:12 +0200]: > >> Hello, >> >> This patch series adds support for the MacVTap device driver in Ganeti. >> >> As stated in the respective design doc, this implementation targets the >> KVM hypervisor. It also includes

You've been removed from the ganeti team Owners role

You’ve been removed from the Owners role team on the ganeti organization. Cheers & Octocats, GitHub Support

GitHub is now the authoritative repo for Ganeti

Hello, Ganeti community! As discussed at GanetiCon last year, we are happy to announce that Ganeti’s GitHub repo [1] is now the authoritative repository for Ganeti. With this change, we are also adapting the development process to use pull requests [2] instead of mailing list patches. We believe

ganeticon.org

Do you want to renew that domain? It's on gandi and about to expire, later this month. Thanks, Guido

Re: [PATCH stable-2.16 0/3] Fix Ganeti memory model on KVM

These patches were extended, refactored, then reviewed and merged using GitHub: https://github.com/ganeti/ganeti/pull/33 https://github.com/ganeti/ganeti/pull/34 https://github.com/ganeti/ganeti/pull/35 On Wednesday, March 22, 2017 at 2:32:48 PM UTC, Viktor Bachraty wrote: > > > [T

[PATCH stable-2.16 0/3] Fix Ganeti memory model on KVM

[This patch is reviewed in parallel on GitHub] Hi, Please review my latest take on fixing ganeti memory model. I've tried to implement the memory model as discussed in [1] and proposed in [2], but without introducing memory overcommitment (swap etc) or dependencies on 2.17 features

[PATCH stable-2.16 2/3] Add optStaticKvmNodeMemory to HTools

value). Signed-off-by: Viktor Bachraty <vbachr...@google.com> --- src/Ganeti/HTools/Backend/Luxi.hs | 13 - src/Ganeti/HTools/CLI.hs | 13 - src/Ganeti/HTools/ExtLoader.hs| 6 +- src/Ganeti/HTools/Loader.hs | 9 + src/

[PATCH stable-2.16 3/3] Redesign Ganeti memory model

to the model proposed in issue 127, but doesn't introduce overcommitment (swap or balooning) support nor does it depend on Ganeti 2.17 features. The goal is to distinguish between reported (state-of-world) values and recorded (state-of-record) values and not to mix them arbitrarily. To estimate free

[PATCH stable-2.16 1/3] Htools should use state-of-record instance size

ing a buffer in the instance's balooning kernel driver. Thus the instance isn't in fact resized, just some pages are borrowed and reused in the free pool. Signed-off-by: Viktor Bachraty <vbachr...@google.com> --- src/Ganeti/HTools/Backend/Luxi.hs | 8 +++- 1 file changed, 3 inserti

Re: [PATCH stable-2.16 1/1] Ceph/RBD rbd showmapped -p is no longer supported

Sorry for the late reply. LGTM, thanks for taking care of this. Morg. On 13 March 2017 at 10:30, 'Viktor Bachraty' via ganeti-devel <ganeti-devel@googlegroups.com> wrote: > From: Ansgar Jazdzewski <a.jazdzew...@googlemail.com> > > In order to solve the missing "-p&

[PATCH stable-2.16 1/1] Ceph/RBD rbd showmapped -p is no longer supported

From: Ansgar Jazdzewski In order to solve the missing "-p" option in "rbd showmapped" it is removed from the showmap_cmd. To make sure we found the volume in the showmapped (json) output, we now check also the pool in the _ParseRbdShowmappedJson method.

[PATCH stable-2.16 1/1] Ceph/RBD rbd showmapped -p is no longer supported

From: Ansgar Jazdzewski In order to solve the missing "-p" option in "rbd showmapped" it is removed from the showmap_cmd. To make sure we found the volume in the showmapped (json) output, we now check also the pool in the _ParseRbdShowmappedJson method.

[PATCH stable-2.16 0/1] Ceph/RBD rbd showmapped -p is no longer supported

Thanks for spotting that. PTAL with fixed unit tests. Ansgar Jazdzewski (1): Ceph/RBD rbd showmapped -p is no longer supported lib/storage/bdev.py | 17 +++-- test/py/ganeti.storage.bdev_unittest.py | 16 +--- 2 files changed, 20 insertions(+), 13

Re: [PATCH stable-2.16] Ceph/RBD rbd showmapped -p is no longer supported

e_function(self.json_output_ok, self.volume_name), TypeError: _ParseRbdShowmappedJson() takes exactly 3 arguments (2 given) Please fix these and also test the extra logic in the new code. On 8 March 2017 at 15:46, 'Viktor Bachraty' via ganeti-devel <ganeti-devel@googlegroups.com> wrote: > From

[PATCH stable-2.16] Ceph/RBD rbd showmapped -p is no longer supported

From: Ansgar Jazdzewski In order to solve the missing "-p" option in "rbd showmapped" it is removed from the showmap_cmd. To make sure we found the volume in the showmapped (json) output, we now check also the pool in the _ParseRbdShowmappedJson method.

Re: [PATCH stable-2.16] Adding .gitignore entry for the maintd binary

ex e653ffc7f..301b0de9e 100644 > --- a/.gitignore > +++ b/.gitignore > @@ -166,6 +166,7 @@ > /src/ganeti-wconfd > /src/ganeti-kvmd > /src/ganeti-luxid > +/src/ganeti-maintd > /src/ganeti-metad > /src/ganeti-mond > /src/rpc-test > -- > 2.12.0.rc1.440.g5b76565f74-goog > >

[PATCH stable-2.16] Adding .gitignore entry for the maintd binary

Signed-off-by: Rafael Marinheiro <marinhe...@google.com> --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index e653ffc7f..301b0de9e 100644 --- a/.gitignore +++ b/.gitignore @@ -166,6 +166,7 @@ /src/ganeti-wconfd /src/ganeti-kvmd /src/ganeti

Re: [PATCH stable-2.16] Fix missing import in RAPI server

Thanks for catching this! LGTM. On Monday, February 20, 2017 at 3:02:09 PM UTC, Yiannis Tsiouris wrote: > > This commit fixes a bug introduced in 7b1fd2e60 after pylint cleanup of > "unused" imports. We also need to suppress this warning as pylint cannot > determine that this import was

Re: [PATCH stable-2.16 1/1] Add discard KVM option to bdev

: Ansgar Jazdzewski <a.jazdzew...@googlemail.com> > > Signed-off-by: Viktor Bachraty <vbachr...@google.com> > > --- > > lib/hypervisor/hv_kvm/__init__.py | 12 ++-- > > man/gnt-instance.rst | 9 + > > src/Ganeti/Constants.hs

Re: [PATCH master] Expose allow-failover option to RAPI client

Hi Alex, Yiannis, Thank you for the patch! LGTM. Viktor On Monday, February 6, 2017 at 2:17:22 PM UTC, Yiannis Tsiouris wrote: > > 'gnt-instance migrate' supports performing a failover on an instance > that is not possible to migrate (e.g. if the instance is down). This > patch exposes this

[PATCH stable-2.16 1/1] Add discard KVM option to bdev

lt;vbachr...@google.com> --- lib/hypervisor/hv_kvm/__init__.py | 12 ++-- man/gnt-instance.rst | 9 +++++ src/Ganeti/Constants.hs | 20 3 files changed, 39 insertions(+), 2 deletions(-) diff --git a/lib/hypervisor/hv_kvm/__init__.py b/lib/hyper

[PATCH stable-2.16] Add dicard KVM option to bdev

this patch will add the discard-options to ganeti (KVM only), it can be helpful if you are using ext-storage (iSCSI) or ceph (RBD) so free space on a SAN is possible diff --git a/lib/hypervisor/hv_kvm/__init__.py b/lib/hypervisor/hv_kvm/__init__.py index 1f4184d..c1838ae 100644 --- a/lib

Re: [PATCH stable-2.14] backend: Use uuid and not idx for bdev symlinks

that dimara's patch never made it to stable-2.14 (and > to the > > rest stable branches), am I mistaken? > > > > Regards, > > Yiannis > > > > [1]: https://groups.google.com/forum/#!topic/ganeti-devel/n6gHGaaFqEg > > Any update on this? :) > > Cheers, > Yiannis >

Re: Issue 1180 in ganeti: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command

Comment #8 on issue 1180 by a.jazdze...@googlemail.com: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command https://code.google.com/p/ganeti/issues/detail?id=1180 as far as i had a look the other sub-commands still have the -p flag -- You received this message

Re: [PATCH stable-2.15] Fix tuple-unpacking from QueryInstances result

Patch has been pushed to stable-2.15 @ 9a62b430e30e8a76df628c36fe8c1787535642c0 On 31 January 2017 at 10:59, Federico Pareschi wrote: > Good catch, the patch looks good to me. I've stumbled upon problems > like this in the past with python, it can be really annoying. I'll be >

Re: [PATCH stable-2.15 2/2] Add test for removing disk on a live instance

Thanks. LGTM. On Thursday, January 26, 2017 at 3:14:25 PM UTC, Yiannis Tsiouris wrote: > > Add a unit test for removing a disk from a running instance without > using hotplug. > > Signed-off-by: Yiannis Tsiouris > Signed-off-by: Filippos Giannakos > ---

Re: [PATCH stable-2.15] Fix tuple-unpacking from QueryInstances result

Good catch, the patch looks good to me. I've stumbled upon problems like this in the past with python, it can be really annoying. I'll be pushing this to 2.15, thanks. On 30 January 2017 at 13:18, Yiannis Tsiouris wrote: > Prior to this commit, some code segments tried to unpack

Re: [PATCH stable-2.15 1/2] Prohibit disk removal w/o hotplug on live instance

Prior to this commit, removing a disk from a live instance without > using hotplug succeeded incorrectly leaving stale disks behind. This is > due to the fact that the hypervisor holds the device open while Ganeti > tries to shut it down. The operation fails but Ganeti continues anyway, > remo

Re: [PATCH stable-2.15] Fix idx in RemoveDisks warning

Patch has been pushed to stable-2.15 @ 48d11548319cfebf5f701c35d02a4d79c0f51bb1 On 31 January 2017 at 10:40, Federico Pareschi wrote: > Hey Yannis. Patch looks good, thanks for submitting this, I'll be > pushing it to 2.15. > > On 26 January 2017 at 21:03, Yiannis Tsiouris

Re: [PATCH stable-2.15] Fix idx in RemoveDisks warning

Hey Yannis. Patch looks good, thanks for submitting this, I'll be pushing it to 2.15. On 26 January 2017 at 21:03, Yiannis Tsiouris wrote: > Before this patch, the warning of RemoveDisks was incorrectly showing > the index of the disk (whose removal failed) in the list of the >

Re: Issue 1180 in ganeti: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command

Comment #7 on issue 1180 by sup...@gmail.com: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command https://code.google.com/p/ganeti/issues/detail?id=1180 Yes, I recently upgraded some servers and now all of the instances on CEPH are broken. I'm trying

Re: Issue 994 in ganeti: Support Setting the UUID of a ganeti instance explicitly add instance creation time

Comment #11 on issue 994 by darius.s...@rkt-online.com: Support Setting the UUID of a ganeti instance explicitly add instance creation time https://code.google.com/p/ganeti/issues/detail?id=994 I did a little mistake in fourth patch. Here comes the right one. Attachments: write-uuid

Re: Issue 994 in ganeti: Support Setting the UUID of a ganeti instance explicitly add instance creation time

Comment #9 on issue 994 by darius.s...@rkt-online.com: Support Setting the UUID of a ganeti instance explicitly add instance creation time https://code.google.com/p/ganeti/issues/detail?id=994 Hello, I use ganeti since 2.11 and have also discovered this "uuid" problem with t

Re: Issue 546 in ganeti: gnt-instance reinstall man page has confusing {instance...} argument in synopsis

Updates: Status: Fixed Comment #4 on issue 546 by m...@google.com: gnt-instance reinstall man page has confusing {instance...} argument in synopsis https://code.google.com/p/ganeti/issues/detail?id=546 So it turns out the change wasn't trivial at all. I've spent some time looking

Re: [PATCH stable-2.15] Make man pages more consistent in parameter format

Well-caught typo, Brian, thanks. I fixed the typo and submitted this @ stable-2.15 970ea7dcab391d4756ed5d7f1c89fcd035ad64ba On Monday, January 16, 2017 at 1:29:48 PM UTC, Brian Foley wrote: > > On Thu, Jan 12, 2017 at 10:59:55AM +, 'Federico Morg Pareschi' via > ganeti-de

Re: [PATCH stable-2.15] Make man pages more consistent in parameter format

On Thu, Jan 12, 2017 at 10:59:55AM +, 'Federico Morg Pareschi' via ganeti-devel wrote: > This patch adds some more consistency in the parameter names > displayed in various ganeti man pages and their helper text > in gnt commands. Yikes. A huge and tedious patch. Thanks for doing t

[PATCH stable-2.15] Make man pages more consistent in parameter format

This patch adds some more consistency in the parameter names displayed in various ganeti man pages and their helper text in gnt commands. Signed-off-by: Federico Morg Pareschi <m...@google.com> --- lib/client/gnt_backup.py | 6 +++--- lib/client/gnt_cluster.py | 21 +

Re: [PATCH stable-2.15] Make man pages more consistent in parameter format

planned it to be, but it's only cosmetic changes and documentation, no functional change. On 4 January 2017 at 12:18, 'Brian Foley' via ganeti-devel <ganeti-devel@googlegroups.com> wrote: > On Tue, Jan 03, 2017 at 09:35:20AM +, Federico Pareschi wrote: >> Interesting, you rai

Re: Issue 1201 in ganeti: ERROR: cluster: Client certificate digest of master candidate

Updates: Status: Invalid Comment #2 on issue 1201 by m...@google.com: ERROR: cluster: Client certificate digest of master candidate https://code.google.com/p/ganeti/issues/detail?id=1201 Hi, I'm glad you managed to fix your problem. I'll be closing this issue as it's not a valid

Re: Issue 1201 in ganeti: ERROR: cluster: Client certificate digest of master candidate

Comment #1 on issue 1201 by tsenden...@gmail.com: ERROR: cluster: Client certificate digest of master candidate https://code.google.com/p/ganeti/issues/detail?id=1201 Hi All, Eventually solved with the following command gnt-cluster renew-crypto --new-cluster-certificate --new-node

Issue 1201 in ganeti: ERROR: cluster: Client certificate digest of master candidate

Status: New Owner: New issue 1201 by tsenden...@gmail.com: ERROR: cluster: Client certificate digest of master candidate https://code.google.com/p/ganeti/issues/detail?id=1201 What software version are you running? Please provide the output of "gnt- cluster --version", &q

Re: Issue 1162 in ganeti: Cannot failover shared storage instances from offline nodes

Comment #3 on issue 1162 by dimitris...@gmail.com: Cannot failover shared storage instances from offline nodes https://code.google.com/p/ganeti/issues/detail?id=1162 Issue 1200 has been merged into this issue. -- You received this message because this project is configured to send all

Re: Issue 1200 in ganeti: gnt-instance failover --ignore-consistency does not work for extstorage

Updates: Status: Duplicate Mergedinto: 1162 Comment #1 on issue 1200 by dimitris...@gmail.com: gnt-instance failover --ignore-consistency does not work for extstorage https://code.google.com/p/ganeti/issues/detail?id=1200 (No comment was entered for this change.) -- You

Issue 1200 in ganeti: gnt-instance failover --ignore-consistency does not work for extstorage

Status: New Owner: New issue 1200 by gannef...@googlemail.com: gnt-instance failover --ignore-consistency does not work for extstorage https://code.google.com/p/ganeti/issues/detail?id=1200 Hi ganeti 2.15 (2.15.2-1~gdn7+1, Debian package) Disks ONLY by extstorage Situation: Multi

Issue 1199 in ganeti: Cannot override --prealloc-wipe-disks at instance level

Status: Accepted Owner: dimitris...@gmail.com Labels: Type-Defect Priority-High Component-storage New issue 1199 by dimitris...@gmail.com: Cannot override --prealloc-wipe-disks at instance level https://code.google.com/p/ganeti/issues/detail?id=1199 If --prealloc-wipe-disks is set at cluster

Re: [PATCH master] Implement predictive queue cluster parameter

onto master then, > thanks for the review. > > On 9 January 2017 at 11:40, 'Viktor Bachraty' via ganeti-devel > <ganeti-devel@googlegroups.com> wrote: >> Thanks for the patch! LGTM, just one minor comment. Personally I'd prefer >> '--enable-predictive-queue' ra

Re: [PATCH master] Implement predictive queue cluster parameter

I agree completely, and the --user-shutdown flag is the reason why I kept it consistent with it. I'll be pushing this onto master then, thanks for the review. On 9 January 2017 at 11:40, 'Viktor Bachraty' via ganeti-devel <ganeti-devel@googlegroups.com> wrote: > Thanks for the patch! L

Re: [PATCH stable-2.15] Re-add 2 imports incorrectly removed during cleanup

Yikes, LGTM. Sorry for not spotting this at review time. Thanks for fixing it. On 9 January 2017 at 11:37, 'Brian Foley' via ganeti-devel <ganeti-devel@googlegroups.com> wrote: > This was done in commit 7b1fd2e60e856ac to quell pylint (my bad!) > > These broke ganeti-rapi and move

Re: [PATCH master] Implement predictive queue cluster parameter

flags (e.g. --user-shutdown). On Monday, January 9, 2017 at 11:18:01 AM UTC, Federico Pareschi wrote: > > This commit adds the enabled_predictive_queue cluster parameter that > allows the cluster to specify if the predictive scheduler should be used > or not when ordering ganeti jobs

[PATCH stable-2.15] Re-add 2 imports incorrectly removed during cleanup

This was done in commit 7b1fd2e60e856ac to quell pylint (my bad!) These broke ganeti-rapi and move-instance. Make sure, as a minimal sanity check, that all the ganeti tools start up with --help. Signed-off-by: Brian Foley <bpfo...@google.com> --- lib/server/rapi.py | 1 + tools/move-in

[PATCH master] Implement predictive queue cluster parameter

This commit adds the enabled_predictive_queue cluster parameter that allows the cluster to specify if the predictive scheduler should be used or not when ordering ganeti jobs in the queue. It also adds the correct command line flags for the cluster init and cluster modify ganeti jobs

Re: [PATCH-2.15] kvm: use_guest_agent: QEMU Guest Agent support

patch for your distro for users who Know What They're Doing. Thanks, Brian. > Signed-off-by: Robin H. Johnson <robb...@gentoo.org> > --- > lib/hypervisor/hv_kvm/__init__.py | 23 +++ > man/gnt-instance.rst | 7 +++ > src/Ganeti/

Re: [PATCH-master v2] kvm: use_guest_agent: QEMU Guest Agent support

; lib/hypervisor/hv_kvm/__init__.py | 23 +++ > man/gnt-instance.rst | 7 +++ > src/Ganeti/Constants.hs | 5 + > 3 files changed, 35 insertions(+) > > This version applies cleanly to 2.17 and 2.16. Thanks for picking this up again

Re: [PATCH stable-2.15] Add status and "version implemented" fields to design docs

This makes investigating particular features much easier. Thanks for this cleanup effort! LGTM. On Wednesday, January 4, 2017 at 4:18:49 PM UTC, Brian Foley wrote: > > Based on a combination of reading NEWS, Ganeti version design docs, and > looking through the commit logs to see when

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Comment #10 on issue 1198 by darius.s...@rkt-online.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 It works now with current stable-2.15, thank you. -- You received this message because this project is configured to send all

Re: [PATCH master] Fix incorrect 'any' function call

Commit has been pushed at ba7013826ab681a5c2b5c1ae7d17dda9cc9d03f6 on master. Thanks. On 4 January 2017 at 15:16, Brian Foley <bpfo...@google.com> wrote: > On Wed, Jan 04, 2017 at 03:11:41PM +, 'Federico Morg Pareschi' via > ganeti-devel wrote: >> As introduced

[PATCH stable-2.15] Add status and "version implemented" fields to design docs

Based on a combination of reading NEWS, Ganeti version design docs, and looking through the commit logs to see when features were implemented. Clean up a number of inconsistencies and missed entries while here, and update a number of (partly) implemented designs as no longer being drafts. Signed

Re: [PATCH master] Fix incorrect 'any' function call

On Wed, Jan 04, 2017 at 03:11:41PM +, 'Federico Morg Pareschi' via ganeti-devel wrote: > As introduced by commit 77807d83 and d388e30c6d > and was not caught during the code review process. LGTM. Thanks.

[PATCH master] Fix incorrect 'any' function call

As introduced by commit 77807d83 and d388e30c6d and was not caught during the code review process. Signed-off-by: Federico Morg Pareschi --- lib/cmdlib/instance_set_params.py | 6 +++--- tools/move-instance | 6 +++--- 2 files changed, 6 insertions(+), 6

Re: [PATCH master] Fix incorrect 'any' function call

ley <bpfo...@google.com> wrote: > > On Wed, Jan 04, 2017 at 02:11:15PM +, 'Federico Morg Pareschi' via > > ganeti-devel wrote: > >> As introduced by commit 77807d831976cf018ace04961820987a91c1d58a > >> and was not caught during the code review process. >

Re: [PATCH master] Fix incorrect 'any' function call

It happens, it escaped my review as well so it's not just your fault. I assume this is an LGTM? :) On 4 January 2017 at 14:38, Brian Foley <bpfo...@google.com> wrote: > On Wed, Jan 04, 2017 at 02:11:15PM +, 'Federico Morg Pareschi' via > ganeti-devel wrote: >> As in

Re: [PATCH master] Fix incorrect 'any' function call

On Wed, Jan 04, 2017 at 02:11:15PM +, 'Federico Morg Pareschi' via ganeti-devel wrote: > As introduced by commit 77807d831976cf018ace04961820987a91c1d58a > and was not caught during the code review process. > > Signed-off-by: Federico Morg Pareschi <m...@google.com> &

Re: [PATCH stable-2.15] Fix typo in cli_opts.py IGNORE_HVVERSIONS_OPT flag

Commit was pushed at 24ca5625813f087963a3f2c768a5a5bd23bfee4c on 2.15 branch. Thanks :) On 4 January 2017 at 12:08, Brian Foley <bpfo...@google.com> wrote: > On Tue, Jan 03, 2017 at 08:46:06AM +, 'Federico Morg Pareschi' via > ganeti-devel wrote: >> Signed-off-by: Federic

[PATCH master] Fix incorrect 'any' function call

As introduced by commit 77807d831976cf018ace04961820987a91c1d58a and was not caught during the code review process. Signed-off-by: Federico Morg Pareschi --- lib/cmdlib/instance_set_params.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: Issue 1180 in ganeti: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command

Comment #6 on issue 1180 by bpfo...@google.com: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command https://code.google.com/p/ganeti/issues/detail?id=1180 Thanks. This patch looks good as far as it goes, but as I mentioned in #3 it doesn't fix the same with all

Re: Issue 1185 in ganeti: hbal: IntMap.!: key -1 is not an element of the map

Updates: Status: Fixed Comment #20 on issue 1185 by bpfo...@google.com: hbal: IntMap.!: key -1 is not an element of the map https://code.google.com/p/ganeti/issues/detail?id=1185 Fixed in http://git.ganeti.org/?p=ganeti.git;a=commit;h=8fcb5ec531c36 on stable-2.15 -- You received

Re: [PATCH stable-2.15] Fix coexistence of location tags and non-DRBD instances

On Mon, Dec 05, 2016 at 12:04:07PM +0100, Iustin Pop wrote: >On 4 December 2016 at 18:44, Brian Foley <[1]bpfo...@google.com> wrote: > > On Fri, Dec 02, 2016 at 11:03:55PM +0100, Iustin Pop wrote: > > From: Iustin Pop <[2]ius...@google.com> > > > > This addresses issue

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Updates: Status: Fixed Comment #9 on issue 1198 by bpfo...@google.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 Fix with commit http://git.ganeti.org/?p=ganeti.git;a=commit;h=081dc6083bdcaa in stable-2.15 -- You received

Re: [PATCH stable-2.15] Fix backup export in case of ext disk template

On Wed, Jan 04, 2017 at 02:01:54PM +0200, Dimitris Aragiorgis wrote: > This commit fixes backup export in case of ext disk template > in two ways: > > - Makes the generated snapshot (if any) inherit the params of the > origin disk. > > - Partially reverts commit 712ea2f that did export the

Re: [PATCH stable-2.15] Make man pages more consistent in parameter format

On Tue, Jan 03, 2017 at 09:35:20AM +, Federico Pareschi wrote: > Interesting, you raise valid points for a patch that I thought was > trivial but indeed is not. > > > Per our discussion yesterday, all the gnt-group commands actually do work > > with > > either group specified by name or by

Re: [PATCH stable-2.15] Fix typo in cli_opts.py IGNORE_HVVERSIONS_OPT flag

On Tue, Jan 03, 2017 at 08:46:06AM +, 'Federico Morg Pareschi' via ganeti-devel wrote: > Signed-off-by: Federico Morg Pareschi <m...@google.com> > --- > lib/cli_opts.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/cli_opts.py b/lib/cli

Re: [PATCH stable-2.15] Make man pages more consistent in parameter format

Interesting, you raise valid points for a patch that I thought was trivial but indeed is not. > Per our discussion yesterday, all the gnt-group commands actually do work with > either group specified by name or by UUID, so this is an improvement to the > docs. However, the command line help in

[PATCH stable-2.15] Fix typo in cli_opts.py IGNORE_HVVERSIONS_OPT flag

Signed-off-by: Federico Morg Pareschi --- lib/cli_opts.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/cli_opts.py b/lib/cli_opts.py index 0b1bb0c..75379aa 100644 --- a/lib/cli_opts.py +++ b/lib/cli_opts.py @@ -852,7 +852,7 @@ IGNORE_CONSIST_OPT =

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Updates: Owner: dimitris...@gmail.com Comment #8 on issue 1198 by dimitris...@gmail.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 Of course this is the commit to blame! When I first looked at it I missed this part

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Comment #7 on issue 1198 by darius.s...@rkt-online.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 Hello Dimitris, I did bisecting. Indeed the first bad (relevant) commit is 712ea2f2e68690cdda4f7be0ecb9808343b8b05c as you stated

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Comment #6 on issue 1198 by darius.s...@rkt-online.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 In the meantime I did more tests, polished my provider log output again (added date) and also added some log output lines inside

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Comment #5 on issue 1198 by darius.s...@rkt-online.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 I don't really understand what you mean by "local" export? The last exports I've tested with your patch where running in

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Comment #4 on issue 1198 by dimitris...@gmail.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 I don't think there are changes between v2.15.2 and stable-2.15 in the extstorage-related code. The only relevant commit is: https

Re: Issue 1055 in ganeti: failed to gnt-backup export

Comment #3 on issue 1055 by star...@gmail.com: failed to gnt-backup export https://code.google.com/p/ganeti/issues/detail?id=1055 I was able to work around this issue by removing the instance's swap disk before export. It seems likely that gnt-backup is just missing logic for volumes

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Comment #3 on issue 1198 by darius.s...@rkt-online.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 Hello Dimitris, I have applied your patch to ganeti 2.15.2 (release) and it works! I checked snapshot activity with "lvs| grep

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Updates: Status: Accepted Labels: Type-Defect Component-storage Comment #2 on issue 1198 by dimitris...@gmail.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 It seems that the params of the snapshot disk

Re: Issue 1180 in ganeti: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command

Comment #5 on issue 1180 by a.jazdze...@googlemail.com: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command https://code.google.com/p/ganeti/issues/detail?id=1180 i put the patch on github https://github.com/ganeti/ganeti/pull/27 -- You received this message

Re: Issue 1180 in ganeti: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command

Comment #4 on issue 1180 by a.jazdze...@googlemail.com: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command https://code.google.com/p/ganeti/issues/detail?id=1180 i had a look into the json output of "rbd showmapped" and i thik i can be done more se

Re: Issue 1198 in ganeti: gnt-backup does not attach ext-based snapshot

Comment #1 on issue 1198 by darius.s...@rkt-online.com: gnt-backup does not attach ext-based snapshot https://code.google.com/p/ganeti/issues/detail?id=1198 Using gnt-backup without "--noshutdown" works, because no snapshot has to be taken. I meant of course an instance

Re: [PATCH stable-2.15] Fix miscellaneous typos found when reading design docs

LGTM, I was planning to go through a lot of these myself when I had the time, so thanks a lot for taking care of it! On 22 December 2016 at 15:29, 'Brian Foley' via ganeti-devel <ganeti-devel@googlegroups.com> wrote: > Some of these are also in code comments, but don't touch any string &

[PATCH stable-2.15] Fix miscellaneous typos found when reading design docs

- lib/client/gnt_job.py| 2 +- man/gnt-cluster.rst | 4 ++-- man/mon-collector.rst| 4 ++-- src/Ganeti/Confd/Server.hs | 4 ++-- src/Ganeti/THH.hs| 2 +- src/Ganeti/WConfd/ConfigState.hs | 2

Re: [PATCH stable-2.15] Make man pages more consistent in parameter format

On Tue, Dec 20, 2016 at 03:12:50PM +, 'Federico Morg Pareschi' via ganeti-devel wrote: > This patch adds some more consistency in the parameter names > displayed in various ganeti man pages. > > Signed-off-by: Federico Morg Pareschi <m...@google.com> > --- > m

[PATCH stable-2.15] Make man pages more consistent in parameter format

This patch adds some more consistency in the parameter names displayed in various ganeti man pages. Signed-off-by: Federico Morg Pareschi <m...@google.com> --- man/gnt-group.rst| 8 man/gnt-instance.rst | 16 man/gnt-job.rst | 6 +++--- man/gnt-netwo

Re: [PATCH master] FIX: Add missing LockDecls files to Makefile.am

On Mon, Dec 19, 2016 at 04:10:08PM +, 'Federico Morg Pareschi' via ganeti-devel wrote: > Trivial change to add the missing LockDecls.hs files to Makefile.am. > This should remove some complaints to some of the tests that failed to > build. > > Signed-off-by: Federico M

[PATCH master] FIX: Add missing LockDecls files to Makefile.am

e.am b/Makefile.am index e18ee7e..dc8b63b 100644 --- a/Makefile.am +++ b/Makefile.am @@ -997,6 +997,7 @@ HS_LIB_SRCS = \ src/Ganeti/JQScheduler/Types.hs \ src/Ganeti/JQueue.hs \ src/Ganeti/JQueue/Lens.hs \ + src/Ganeti/JQueue/LockDecls.hs \ src/Ganeti/JQueue/Obje

  1   2   3   4   5   6   7   8   9   10   >