Status: New
Owner: ----

New issue 864 by [email protected]: gnt-instance info backend "memory" value should have been removed
http://code.google.com/p/ganeti/issues/detail?id=864

[ganeti 2.10.5]

# gnt-instance info netdot14.ws.nsrc.org | grep mem    mem_path: default ()
    maxmem: default (512)
    memory: default (512)    << THIS ONE
    minmem: default (256)

Looking at git head, it's just a duplicate of "maxmem", and it was intended to be removed ages ago:

[lib/client/gnt_instance.py]

  # deprecated "memory" value, kept for one version for compatibility
  # TODO(ganeti 2.7) remove.
  be_actual = copy.deepcopy(instance["be_actual"])
  be_actual["memory"] = be_actual[constants.BE_MAXMEM]

I think retaining it is just leading to confusion over what it actually means.

Related:
#850 ('memory' setting has already been removed from config.ini)
#695 (includes discussion over what the 'memory' setting actually means)

Note: it's still possible to set -B memory for backwards compatibility, and this can stay. AFAICS it just sets both minmem and maxmem (except doesn't work when you set it to "default": see #863)


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

Reply via email to