Revert "proxmox: add SysError::is_errno_raw"

Not actually required, the numbers just overlapped and are
therefore no separate enum entry.

This reverts commit 8f57f29ca84dbfdb99a822fdfbba43b361577a81.

proxmox: add SysError::is_errno_raw

to work around missing errno values in nix 0.16

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fixup last commit

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox: tools: add RawFdNum helper

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api/cli/completion.rs: implement completion for lists of strings

IP_REGEX - match whole string

bump proxmox crate to 0.1.22-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox-api-macro to 0.1.7-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox-sortable-macro to 0.1.1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

switch from failure to anyhow

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox-api-macro/src/api/method.rs: fix error message

api-macro: add test for access w/o a description

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: avoid duplicate output code

While this is a tiny piece of code, having only one output
code consisting of prepared parts is nicer to keep up to
date.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: cleanup & test fixup

remove unnecessary types, they were already in the struct
update test to use Option for access description

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump rust-proxmox-api-macro version to 0.1.6-1

bump rust-proxmox version to 0.1.21-1

proxmox-api-macro/tests/api1.rs: fix test case

proxmox/src/api/permission.rs: pass parameters as HashMap

- to make it suitable for our rest server...
- also avoid repeated is_superuser() checks
- fix privilege test (add check for 0)

make access description optional

api macro: support reload_timezone

list_subdirs_api_method: allow access for Anybody

style cleanup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rename some permission/access items

router.permissions(...) -> router.access(...)
    to be more consistent with the other builder methods and
    struct member names

ApiAccessPermissions -> ApiAccess
    shorter, not necessarily with defined permissions, and
    gets rid of a singular/plural confusion

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: support 'access' specification for methods

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: error on unexpected keys

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: util: add join and join_debug helper

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: minor cleanup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api/permissions: support test-harness

(Api-macro tests want to use `==` to compare generated
schemas against hardcoded schemas.)

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api/permissions: cleanup & formatting

Use the `write!` macro as a shortcut for
   f.write_fmt(format_args!(...))

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

flatten use hierarchy for api::Permissions

implement access permission system

bump proxmox crate to 0.1.20-1

improve command line completion with optional argument parameters

api-macro: started with some more documentation

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: add consts for defaults

Not sure how we want to actually handle default values yet.
For now we need to figure out what "feels best".
This adds default values as publicly available consts so
they can be accessed from anywhere as
`API_METHOD_<METHOD_NAME>_PARAM_DEFAULT_<PARAM_NAME>`.
This is quite a handful to type, but I'm not sure how to
make this better.

Within the function body we also have the
`api_get_default!(param_name)` macro which is more
convenient, but not available outside the `#[api]` tagged
function.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: explicitly enable syn's visit-mut feature

otherwise the build fails because the module is not visible

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

api-macro: add api_get_default!() macro

When writing an #[api] function, one can now access default
values by parameter name (see test_default_option in
tests/options.rs):

    #[api(...)]
    pub fn func(value: Option<isize>) {
        println!(
            "value: {}",
            value.unwrap_or(api_get_default!("value")),
        );
    }

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: tests for optional non-Option parameters

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: allow optional types without Option<T>

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox crate to 0.1.19-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump api-macro to 0.1.5

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

make cli format a bit saner

this looks more like pve/pmg cli tools now
without double the empty lines

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

buildsys: drop format check in check target

rename 'checkfmt' target to 'fmt'
there's no "apply formatting" target anymore now, this has
to be done manually to not do this by accident

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: cleanup: warnings

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: cleanup: don't use try_fold for a Vec

`collect()` uses `FromIterator` which is also implemented
for `Result<Vec, Err>` from `Result<Item, Err>` doing the
same thing.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: add integer limit test cases

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: include default minimum/maximum for integer types

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: replace unreachable with a panic

(we're not running into this, but ran into an unreachable in
`syn` during development, and I needed to make sure it's not
one of ours...)

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: correctly infer newtype types

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: factor handle_struct differently

Don't assume an empty object schema for all cases as
newtypes shouldn't use an object schema at all actually!

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

build: add -upload target

requires 'dcmd' from devscripts to extract binary package file names
from .changes file

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

Revert "add proxmox/src/sys/linux/fs.rs"

This reverts the previous commit 21b9321cf81409cb07ef332076f54dc4f32141ce.

Turn out we only need this for pxar, so this will get moved into the pxar crate.

add proxmox/src/sys/linux/fs.rs

bump rust-proxmox version 0.1.18-1

src/api/section_config.rs: moved SectionConfig from proxmox-backup

fix typo in doc comment

bump rust-proxmox version 0.1.17-1

text_table.rs: fix max column width when displaying without header

text_table.rs: add table padding

text_table.rs: fix column seperator for ascii tables

bump proxmox to 0.1.16

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox-api-macro to 0.1.4

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api-macro: make API_{RETURN,PARAMETER}_SCHEMA_...  public

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump rust-proxmox version to 0.1.15-1

proxmox/src/api/cli/text_table.rs: new helpers to read output format from environment

bump rust-proxmox version to 0.1.14-1

proxmox/src/api/cli/text_table.rs - format_object: use column_config header

Also right_align Value column if all properthies are right_aligned.

proxmox/src/api/cli/text_table.rs: implement sort by multiple columns

Also implement sort order flag and column header property.

src/api/schema.rs - verify_json_object: improve error message

bump librust-proxmox version to 0.1.13-1

src/api/cli/text_table.rs: use new sys::linux::tty::stdout_terminal_size

src/sys/linux/tty.rs: add tty helpers

fix typos in doc comments.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

bump rust-proxmox version to 0.1.12-1

src/api/cli/text_table.rs: only call TIOCGWINSZ on terminals

src/api/cli/text_table.rs - TableFormatOptions: allow to add single column configs

bump version to 0.1.11

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox: bump api-macro version dependency

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

formatting fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump api-macro to 0.1.3

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

expose parameter schema as API_PARAMETER_SCHEMA_<methodname>

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

expose return schema as API_RETURN_SCHEMA_<methodname>

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump rust-proxmox version to 0.1.10-1

proxmox/src/api/cli/text_table.rs: new table formatter

rust-proxmox: bump version to 0.1.9-1

api/cli/command.rs: implement both sync and async versions.

src/api/cli/command.rs - handle_nested_command: fix help output

fix denglisch: s/rest/remaining/

"rest" as word means "to relax" in english, not the "remaining
stuff", so swap rest out with remaining/remainder, respectively.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump proxmox to 0.1.8 to depend on api macro 0.1.2

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump api-macro

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fix property sorting

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sys: doc update

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sys: hide io_assert macro for now

too risky since errno-changing calls before it can ruin the
result

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

build fixups

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

remove valgrind feature for now and bump to 0.1.7

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

Merge remote-tracking branch 'proxdev/master'

merge api, sys and tools into proxmox directly

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

Add make dinstall target

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

formatting fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox-api version