On Tue, Oct 02, 2012, Pitucha, Stanislaw Izaak wrote:
> Does anyone know why soft-delete is still in place?
> Are there any reasons it can't / shouldn't be removed at this time?
> If it's possible to remove it, would you miss it?
I'm certainly not a fan of the database soft-delete for many of the
On Tue, Oct 02, 2012, Pitucha, Stanislaw Izaak wrote:
> I'd like to open a discussion on a topic that's been bugging me for a number
> of reasons - soft deletes (by that I mean marking rows with deleted=1 in the
> db) and related - actions audit.
This is a subject I'm interested in and I'll follo
On Tue, Jul 31, 2012, Bhuvaneswaran A wrote:
> If a subscriber reply to a mailing list message, it's sent to the
> author only. Each subscriber should use "Reply to All" every time, to
> post a reply to mailing list.
>
> Can you please configure the mailing list and set reply-to header as
> maili
On Fri, Jul 20, 2012, Vishvananda Ishaya wrote:
> When I was going through the list of reviewers to see who would be good
> for nova-core a few days ago, I left one out. Sean has been doing a lot
> of reviews lately[1] and did the refactor and cleanup of the driver
> loading code. I think he would
On Wed, Jul 18, 2012, Vishvananda Ishaya wrote:
> Michael wrote the image cache management code, did all of the remaining
> conversions of instance_id -> instance_uuid, and has been contributing
> a lot to reviews[1]. I think he would make a great addition to nova-core.
+1!
JE
On Wed, Jul 18, 2012, Vishvananda Ishaya wrote:
> Yun has been putting a lot of effort into cleaning up our state
> management, and has been contributing a lot to reviews[1]. I think he
> would make a great addition to nova-core.
+1!
JE
___
Mailing l
On Wed, Jul 18, 2012, Vishvananda Ishaya wrote:
> Padraig has been contributing a lot of code to all parts of nova, and
> has been contributing a lot to reviews[1]. I think he would make a
> great addition to nova-core.
+1!
JE
___
Mailing list: https
On Tue, Jul 03, 2012, Daniel P. Berrange wrote:
> > It seems to me that we're just as likely to have a review slip through
> > that uses /tmp insecurely as a review slipping through that uses /tmp at
> > all.
>
> We already run a bunch of PEP8 checks across the code on every
> commit. It ought to
On Mon, Jul 02, 2012, Salvatore Orlando wrote:
> I do not see any obvious reason for these failures, especially as they
> appear to occur when the vdi is created. If I recall it correctly, that
> code is stubbed out for unit tests, and it does not seem your patch
> un-stubs it.
It's not when the
On Mon, Jul 02, 2012, Daniel P. Berrange wrote:
> On Mon, Jul 02, 2012 at 08:17:08AM -0700, Johannes Erdfelt wrote:
> > Not using /tmp for large files is a good reason for practical reasons
> > (distributions moving to ramfs for /tmp).
> >
> > But please don't sta
On Mon, Jul 02, 2012, Daniel P. Berrange wrote:
> In Fedora 18, /tmp is going to be a RAM filesystem, so we absolutely
> must not create any sizeable files on /tmp.
>
> In addition from a security POV, we must aim to *never* use /tmp for
> anything at all
>
> http://danwalsh.livejournal.com/11
On Thu, Jun 28, 2012, Daniel P. Berrange wrote:
> On Thu, Jun 28, 2012 at 09:21:20AM -0700, Johannes Erdfelt wrote:
> > What is the Signed-off-by tag used for?
> >
> > Your examples have yourself, but isn't that kind of implied by
> > submitting the patch for
On Thu, Jun 28, 2012, Joshua Harlow wrote:
> I remember hearing once that someone had a openstack import/hacking
> style checking tool.
>
> I was wondering if such a thing existed to verify same the openstack
> way of doing imports and other special checks to match the openstack
> style.
>
> I k
First off, I wanted to say I think these are a great set of
recommendations.
On Wed, Jun 27, 2012, Daniel P. Berrange wrote:
> Fixes: bug #1003373
> Implements: blueprint libvirt-xml-cpu-model
> Change-Id: I4946a16d27f712ae2adf8441ce78e6c0bb0bb657
> Signed-off-by: Daniel P. Berran
On Tue, Jun 12, 2012, Eric Windisch wrote:
> > For instance, an instance migration can take a while since we need to
> > copy many gigabytes of disks to another host. If we want to do a
> > software upgrade, we either need to wait a long time for the migration
> > to finish, or we need to restart
On Tue, Jun 12, 2012, Eric Windisch wrote:
> We actually do have ACKs in ZeroMQ, as far as I understand how they work
> in AMQP, but they're really simple. The send() method is actually
> synchronous with the message being received on the other end. However,
> we don't wait for this and spawn an e
As part of a patch to add idempotency to the xenapi driver, I've
modified impl_kombu driver to implement delayed ACKs. This is so RPC
messages are retried after nova-compute restarts.
https://review.openstack.org/#/c/7323/
However, this only works with impl_kombu.
impl_qpid does not ACK anything
On Fri, Jun 08, 2012, Brian Waldon wrote:
> Can you explain why?
Not to speak for Kiall, but I'll make the argument that there is no
value in treating "admin", "Admin", and "admIn" separately. It can only
lead to confusion and frustration of the people who use OpenStack.
These values are intende
On Thu, Jun 07, 2012, Roman Sokolkov wrote:
> Why openstack doesn't use RPC for resize and migrate stuff?
Nova does use RPC for resizes and migrations.
The SSH keys are for tranferring the instance image data, which would be
inappropriate for RPC.
I think Nova would be open to other means of bu
On Wed, Jun 06, 2012, Gabriel Hurley wrote:
> Stored timestamps should always be in UTC, however efforts should be
> made to support local timezones as a user-configurable option. Horizon
> will be working towards this goal since Django has good timezone
> support, I'd encourage other projects to
On Tue, Jun 05, 2012, James E. Blair wrote:
> One important difference is that the new system does not have
> "retrigger" buttons in Jenkins. If the gate tests fail with a false
> negative, you'll need to leave another "Approved" vote in Gerrit.
This part of your post didn't immediately register
There appears to have been some changes to Jenkins recently. Jobs do not
appear to be associated with Gerrit changes anymore.
https://jenkins.openstack.org/job/gate-nova-python27/
You can see jobs 335 and older have a link to Gerrit, but new jobs don't
have any.
This makes it harder to see what
On Wed, Jun 06, 2012, livemoon wrote:
> I found nova use "utcnow" to get time and write it to db.
> So the create_time of vm also show utc time rather than localtime.
You really don't want to do this. For instance, the database won't store
the timezone, so in places where there is daylight saving
On Mon, Jun 04, 2012, James E. Blair wrote:
> On 06/04/2012 05:47 PM, Gabe Westmaas wrote:
> >Should we revert this change till we get it cleared up?
>
> Here's a proposal to do that:
>
> https://review.openstack.org/#/c/8166/
I approved it because of how bad Jenkins is right now, but it prob
On Thu, May 31, 2012, Mark McLoughlin wrote:
> On Thu, 2012-05-31 at 08:58 -0700, Johannes Erdfelt wrote:
> > My XenAPI idempotency branch delays ACKs until after it's done
> > processing the message to ensure we get the message again after
> > restart.
> >
>
On Thu, May 31, 2012, Mark Washenberger wrote:
> * not be tied up with eventlet on the consumer side
Yes!
> * let the consumer code decide when to ack a message
>(although maybe the concept of acking doesn't exist for all
> implementations?)
My XenAPI idempotency branch delays ACKs until
On Wed, May 09, 2012, Robert Kukura wrote:
> I'm very concerned about the potential of this XenServer/XCP requirement
> to interfere with making the various Quantum agents first-class
> OpenStack services by utilizing current and future openstack-common
> facilities for configuration, logging, DB,
On Mon, Apr 30, 2012, Dan Prince wrote:
> The primary benefit here is it is simply less code to maintain:
>
> The old migrations scripts for Essex are around 6200 lines of code.
>
> The new compacted migration for Essex is around 950 lines of code.
It seems like you're counting raw lines, which
On Fri, Apr 27, 2012, Monsyne Dragon wrote:
> Even better, what would it take to try using Alembic?
> (http://alembic.readthedocs.org/en/latest/front.html#project-homepage)
>
> It's a big improvement over sqlalchemy. Amongst other things,
> migrations are not numbered, they are linked by dependa
On Fri, Apr 27, 2012, Dan Prince wrote:
> > Mirations don't appear to be particularly slow right now, and it
> > doesn't
> > appear that merging migrations will make them significantly faster.
> >
> > What exactly is the benefit of doing this?
>
> Speed wasn't the primary motivation here I suppo
On Thu, Apr 26, 2012, Dan Prince wrote:
> The OpenStack Essex release had 82 database migrations. As these grow
> in number it seems reasonable to clean house from time to time. Now
> seems as good a time as any.
Mirations don't appear to be particularly slow right now, and it doesn't
appear that
On Fri, Mar 23, 2012, Debo Dutta (dedutta) wrote:
> +1 to DBs being slow. But what if we used a combo of memcache and db. Or
> use couch/mongo.
>
> Comparision:
> http://kkovacs.eu/cassandra-vs-mongodb-vs-couchdb-vs-redis
>
> Anyone has experience on large deployments to see the kind of db tra
On Fri, Mar 23, 2012, Kevin L. Mitchell wrote:
> On Fri, 2012-03-23 at 13:43 +, Gabe Westmaas wrote:
> > However, I kind of expect that many users
> > will still poll even if they know they won't get new data until X
> > time.
>
> I wish there was some kind of way for us to issue push notifi
On Thu, Mar 22, 2012, Kevin L. Mitchell wrote:
> On Thu, 2012-03-22 at 11:22 -0500, Andrew Bogott wrote:
> > Nova, this:
> >
> >A docstring ends with an empty line before the closing quotations.
>
> Huh? I thought I removed that...
You did:
https://github.com/openstack/nova/commit/35e98d
On Thu, Mar 22, 2012, Andrew Bogott wrote:
> I propose that this unified style guide be copied into each of
> the above projects, with a mandate to maintain consistency
> henceforth. Any objections?
I recently saw a suggestion to put it into openstack-common, but I don't
remember who made it
On Fri, Mar 16, 2012, Vishvananda Ishaya wrote:
> Now that we have the temdir context manager I was thinking something like:
>
> diff --git a/nova/utils.py b/nova/utils.py
> index e375f11..a3ac896 100644
> --- a/nova/utils.py
> +++ b/nova/utils.py
> @@ -61,9 +61,11 @@ ISO_TIME_FORMAT = "%Y-%m-%dT
On Tue, Mar 13, 2012, Lorin Hochstein wrote:
> As long as we're talking about review issues, I'd really like to have a
> set of reviewing guidelines. Even better would be to have them in
> checklist form to make sure we don't miss things as we do reviews
> (checklists are great: http://gawande.com
On Tue, Mar 13, 2012, Yun Mao wrote:
> There are two places in the current master branch that use tpool:
> NWFilterFirewall and XenAPISession. Are they safe?
I've looked at XenAPISession and it appears to be safe. It doesn't use
logging nor any other locks.
It does use other Python modules, but
On Tue, Mar 13, 2012, Mark McLoughlin wrote:
> On Mon, 2012-03-12 at 17:57 -0500, Josh Kearney wrote:
> > Is this is really a problem that needs solving? I'd like to believe that no
> > member of Nova Core would approve something that they aren't familiar with.
>
> I think the issue is more a cas
On Mon, Mar 12, 2012, Yun Mao wrote:
> My understanding is that if the answer to question3 is yes, then the
> blocking call should be executed in tpool, although it's more likely
> to have bugs in that case.
Please be very careful with tpool. If the code being executed in the
tpool thread ends up
otifications
> between Nova and Glance. I believe the same developer (Johannes
> Erdfelt) did them both. And I think his plan was to put the code
> into the openstack-common library.
>
> Johannes, thoughts?
I only did the work for Glance and it was based on the Nova code at the
time.
On Thu, Mar 08, 2012, Andrew Bogott wrote:
> 1) DB support
>
> I need a database table to keep track of some filesystem
> metadata. My current implementation adds the table via
> nova/db/sqlalchemy/migrate_repo... but is it really necessary to
> coordinate this table with the rest of Nova?
On Mon, Mar 05, 2012, Pádraig Brady wrote:
> File "/usr/lib64/python2.7/threading.py", line 446, in __init__
> self.__daemonic = self._set_daemon()
> File "/usr/lib64/python2.7/threading.py", line 470, in _set_daemon
> return current_thread().daemon
> AttributeError: '_GreenThread' obj
On Fri, Mar 02, 2012, Duncan McGreggor wrote:
> On Fri, Mar 2, 2012 at 2:40 PM, Johannes Erdfelt wrote:
> > Twisted has a much harder programming model with the same blocking
> > problem that eventlet has.
>
> Like so many things that are aesthetic in nature, the statement
On Fri, Mar 02, 2012, Armando Migliaccio
wrote:
> I agree, but then the whole assumption of adopting eventlet to simplify
> the programming model is hindered by the fact that one has to think
> harder to what is doing...Nova could've kept Twisted for that matter.
> The programming model would hav
On Wed, Feb 29, 2012, Yun Mao wrote:
> Thanks for the explanation. Let me see if I understand this.
>
> 1. Eventlet will never have this problem if there is only 1 OS thread
> -- let's call it main thread.
In fact, that's exactly what Python calls it :)
> 2. In Nova, there is only 1 OS thread u
On Wed, Feb 29, 2012, Joshua Harlow wrote:
> Just a thought I was having, that others might want to chime in on.
>
> Has there been any thinking around only using eventlet/greenlet for
> webserver endpoints and using something like multiprocessing for
> everything else?
>
> I know its a fundamen
On Wed, Feb 29, 2012, Yun Mao wrote:
> we sometimes notice this error message which prevent us from starting
> nova services occasionally. We are using a somewhat modified diablo
> stable release on Ubuntu 11.10. It may very well be the problem from
> our patches but I'm wondering if you guys have
On Wed, Feb 29, 2012, Vishvananda Ishaya wrote:
> We have had a memory leak due to an interaction with eventlet for a
> while that Johannes has just made a fix for.
>
> bug:
> https://bugs.launchpad.net/nova/+bug/903199
>
> fix:
> https://bitbucket.org/which_linden/eventlet/pull-request/10/monke
On Wed, Feb 08, 2012, Dugger, Donald D wrote:
> Unless they've changed things (I've worked on so many versions of
> Linux/Unix I do get confused :) but `stdio' is supposed to check and,
> if the output is a file then it's buffered and, if it's a terminal,
> then it's unbuffered.
Hi Don!
I'm pret
On Sat, Dec 31, 2011, Thomas Goirand wrote:
> Here's the output (reformatted by hand to fit in this mail):
>
> root@GPLHost:os>_ # glance index
> ID Name Disk Format Size
> lucid_ami ami 524288000
> kernelaki 4099360
> ramdisk ari 7988037
That's goo
On Sat, Dec 31, 2011, Thomas Goirand wrote:
> On 12/31/2011 02:03 AM, Johannes Erdfelt wrote:
> > On Sat, Dec 31, 2011, Thomas Goirand wrote:
> >> 2011-12-30 17:38:59,348 DEBUG nova.virt.xenapi.vmops [-] Resizing up VDI
> >> 1aacb195-9301-410b-989f-49bdce3a4813 from
On Sat, Dec 31, 2011, Thomas Goirand wrote:
> 2011-12-30 17:38:59,348 DEBUG nova.virt.xenapi.vmops [-] Resizing up VDI
> 1aacb195-9301-410b-989f-49bdce3a4813 from 0GB to 20GB from (pid=14409)
> _resize_instance /usr/lib/python2.7/dist-packages/nova/virt/xe
This is likely the problem. The initial
On Thu, Dec 29, 2011, Thomas Goirand wrote:
> What am I doing wrong? What's that plugin thing about? Note that I've
> put stuff in /etc/xapi.d, like the "xenhost" python script, but it
> doesn't seem that's enough.
Did you place it in /etc/xapi.d or /etc/xapi.d/plugins? The latter is
the correct
On Thu, Dec 15, 2011, Mark Washenberger wrote:
> 1. regular objects (which is what I thought Kevin was talking about. . .
> maybe?)
I think this has been poorly defined so far. I've seen some quick
proposals that include moving instance actions to an Instance class,
etc.
It's hard to make a goo
On Thu, Dec 15, 2011, Kevin L. Mitchell wrote:
> 2. However, I violently disagree with the idea that the DB layer
> must return dicts. It does not, even if you start talking about
> allowing use of other kinds of databases. We can, and should,
> wrap these things in
On Fri, Sep 23, 2011, Salvatore Orlando wrote:
> This appears to have been fixed now.
It's been mostly fixed. However, it appears that the last commit to
lp:nova (which was a branch of mine) has been lost in the move to github.
It's not in the git tree and it's not in lp:nova anymore. It is in
lp
On Wed, Sep 07, 2011, Sandy Walsh wrote:
> ... and that's only from my first few days using Gerrit.
I'd also like to add that the when merges fail, it's not easy to figure
out why.
I had a proposed branch the was approved and then failed to merge. I
received a handful of emails (4?) that were m
On Wed, Sep 07, 2011, Monty Taylor wrote:
> Part of this also comes from a semantic difference in how github and
> gerrit view the world. On github, you develop on your personal fork, and
> then you submit one of the branches in your fork to be pulled - so the
> unit of review is the branch- meani
On Wed, Sep 07, 2011, Jay Pipes wrote:
> The problem is that instead of spending time coding on features and
> bugs for Nova, Glance, Swift and Keystone, a bunch of devs are instead
> spending time working on an alternate solution to what has already
> been decided by the PPB, discussed publicly,
On Wed, Sep 07, 2011, Jay Pipes wrote:
> On Wed, Sep 7, 2011 at 12:24 PM, Johannes Erdfelt
> wrote:
> > Why do core members have that "merge and close" option? Wouldn't it make
> > more sense to restrict that to the Jenkins account?
> >
> > I st
On Wed, Sep 07, 2011, Jay Pipes wrote:
> On Wed, Sep 7, 2011 at 11:34 AM, Sandy Walsh
> wrote:
> > But yes, there is a risk that a core member could just hit "merge and
> > close" and break trunk. That's perhaps the only real "con" I can think of.
>
> That's the entire point of gerrit and a ga
On Fri, Aug 26, 2011, Jesse Andrews wrote:
> I disagree. I find lots of valuable in github even if trunk merges
> require gerrit.
>
> Teams can (and IMHO should) use pull requests to improve the quality
> before it is proposed to trunk. Pull requests to branches can be made
> while the work is
I'm struggling to find a reason why the Openstack project is using
github.
The use of Gerrit and other tools has reduced github to being just a
pretty way to view a git repository.
We can't use the github workflow of forking the repository, since that
confuses the tools for Gerrit. There's workar
On Mon, Aug 22, 2011, James E. Blair wrote:
> If you have any problems or questions, feel free to respond here, talk
> to mtaylor or jeblair on IRC, file a bug at
> https://launchpad.net/openstack-ci>, or submit a patch to
> https://github.com/openstack/openstack-ci>.
In my experience, these chan
On Mon, Aug 15, 2011, Vishvananda Ishaya wrote:
> I pushed a new version of that branch with the suggested change. The
> recent change to our db library to make it use eventlet is stealing the
> stack trace, so we can't reraise the same exception. The branch now
> logs the exception and returns
66 matches
Mail list logo
