]>
git.proxmox.com Git - proxmox-backup.git/log
Dietmar Maurer [Fri, 6 Nov 2020 15:32:35 +0000 (16:32 +0100)]
Remove reference to backup@pam
Dylan Whyte [Fri, 6 Nov 2020 14:46:30 +0000 (15:46 +0100)]
Add screenshots
For:
- api tokens
- new user management interface
- updatae server administration
Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>
Dylan Whyte [Fri, 6 Nov 2020 14:46:28 +0000 (15:46 +0100)]
maintenance: add verification and prune to section
Includes new screen shots of interface
Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>
Dylan Whyte [Fri, 6 Nov 2020 14:46:27 +0000 (15:46 +0100)]
Update where to find certain items since GUI update
- Sync jobs in datastore
- "User management" is now section of Access Control
Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>
Dylan Whyte [Fri, 6 Nov 2020 14:46:26 +0000 (15:46 +0100)]
datastore: description of new datastore view
- Add screenshots from new datastore view
- Add description of comment field in create datastore window
- Add description of each tab in the datastore panel
- Update instructions to add datastore from GUI
Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>
Dylan Whyte [Fri, 6 Nov 2020 14:46:25 +0000 (15:46 +0100)]
backup-client: add section on change-owner command
Add section "Changing the Owner of a Backup Group"
Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>
Dylan Whyte [Fri, 6 Nov 2020 14:46:24 +0000 (15:46 +0100)]
installation & gui: Formatting fixup
Fix some minor formatting errors in the docs
Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>
Wolfgang Bumiller [Fri, 6 Nov 2020 13:55:34 +0000 (14:55 +0100)]
Vec::new is Vec's default default
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Fabian Grünbichler [Fri, 6 Nov 2020 13:44:44 +0000 (14:44 +0100)]
client: properly complete new-owner
with remote Authids, not local Userids.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Fabian Grünbichler [Fri, 6 Nov 2020 12:01:56 +0000 (13:01 +0100)]
www: add AuthidSelector
similar to TokenSelector, but with different fields / mapping of data.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Fri, 6 Nov 2020 10:23:09 +0000 (11:23 +0100)]
tasks: allow access to job tasks
if the user/token could have either configured/manually executed the
task, but it was either executed via the schedule (root@pam) or
another user/token.
without this change, semi-privileged users (that cannot read all tasks
globally, but are DatastoreAdmin) could schedule jobs, but not read
their logs once the schedule executes them. it also makes sense for
multiple such users to see eachothers manually executed jobs, as long as
the privilege level on the datastore (or remote/remote_store/local
store) itself is sufficient.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Fri, 6 Nov 2020 10:23:08 +0000 (11:23 +0100)]
verify: allow unprivileged access to admin API
which is the one used by the GUI.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Fri, 6 Nov 2020 10:23:07 +0000 (11:23 +0100)]
verify: fix unprivileged verification jobs
since the store is not a path parameter, we need to do manual instead of
schema checks. also dropping Datastore.Backup here
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Dietmar Maurer [Fri, 6 Nov 2020 11:12:59 +0000 (12:12 +0100)]
ui: improve prune simulator layout
Fabian Ebner [Tue, 3 Nov 2020 14:28:42 +0000 (15:28 +0100)]
include prune simulator in build
Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>
Fabian Ebner [Tue, 3 Nov 2020 14:28:41 +0000 (15:28 +0100)]
create prune simulator
A stand-alone ExtJS app that allows experimenting with different backup
schedules and prune parameters.
The HTML for the documentation was taken from the PBS docs and adapted to the
context of the simulator.
For performance reasons, the week table does not use
subcomponents, but raw HTML.
Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
Dietmar Maurer [Fri, 6 Nov 2020 07:07:02 +0000 (08:07 +0100)]
ui: require owner for sync jobs
Dietmar Maurer [Fri, 6 Nov 2020 06:40:09 +0000 (07:40 +0100)]
ui: use pbsUserSelector for BackupGroupChangeOwner
Dylan Whyte [Tue, 3 Nov 2020 08:30:54 +0000 (09:30 +0100)]
gui: Add button for changing backup group owner
Extension of fix #2847
Adds an action button to the datastore content view,
to change the owner of a backup.
Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>
Dietmar Maurer [Thu, 5 Nov 2020 13:03:15 +0000 (14:03 +0100)]
get rid of backup@pam
Thomas Lamprecht [Fri, 6 Nov 2020 07:24:30 +0000 (08:24 +0100)]
fixup: acutally commit all changes..
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Fri, 6 Nov 2020 07:11:06 +0000 (08:11 +0100)]
ui: move datastore related files into own folder
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Fabian Grünbichler [Thu, 5 Nov 2020 11:12:24 +0000 (12:12 +0100)]
www: add remote store selector
(hopefully) improved upon NFS export selection in PVE
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Thu, 5 Nov 2020 11:12:23 +0000 (12:12 +0100)]
api: refactor remote client and add remote scan
to allow on-demand scanning of remote datastores accessible for the
configured remote user.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Thu, 5 Nov 2020 11:12:22 +0000 (12:12 +0100)]
types: extract DataStoreListItem
for reuse in remote scan API call
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Dietmar Maurer [Thu, 5 Nov 2020 10:32:59 +0000 (11:32 +0100)]
config: make notify a property string
For example "gc=never,verify=always,sync=error".
Dominik Csapak [Wed, 4 Nov 2020 12:09:38 +0000 (13:09 +0100)]
tools/daemon: fix reload with open connections
instead of await'ing the result of 'create_service' directly,
poll it together with the shutdown_future
if we reached that, fork_restart the new daemon, and await
the open future from 'create_service'
this way the old process still handles open connections until they finish,
while we already start a new process that handles new incoming connections
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Wolfgang Bumiller [Thu, 5 Nov 2020 09:32:48 +0000 (10:32 +0100)]
pxar: more concise EOF handling
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Dominik Csapak [Thu, 5 Nov 2020 08:29:06 +0000 (09:29 +0100)]
pxar/create: handle ErrorKind::Interrupted for file reads
they are not an error and we should retry the read
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Dominik Csapak [Thu, 5 Nov 2020 08:17:43 +0000 (09:17 +0100)]
pxar/create: fix endless loop for shrinking files
when a file shrunk during backup, we endlessly looped, reading/copying 0 bytes
we already have code that handles shrunk files, but we forgot to
break from the read loop
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 18:13:04 +0000 (19:13 +0100)]
bump version to 0.9.6-1
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 18:07:07 +0000 (19:07 +0100)]
ui: sync jobs: add tooltip for remove vanished
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 18:05:46 +0000 (19:05 +0100)]
ui: sync jobs: reorder fields
group local ones togeteher on the left side, and source + schedule
on the right side.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 18:04:42 +0000 (19:04 +0100)]
ui: sync jobs: fix originalValue of owner and improve label
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 18:04:06 +0000 (19:04 +0100)]
ui: sync jobs: only set default schedule when creating new jobs
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Dominik Csapak [Wed, 4 Nov 2020 14:35:08 +0000 (15:35 +0100)]
ui: add shell panel under administration
some users prefer an inline console
we still have the pop-out console in 'Administration'
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 16:15:28 +0000 (17:15 +0100)]
proxy: also rotate auth.log file
no need for triggering re-open here, we always re-open that file.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 15:45:12 +0000 (16:45 +0100)]
proxy: logrotate: do not serialize sending async log-reopen commands
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 15:12:13 +0000 (16:12 +0100)]
api: factor out auth logger and use for all API authentication failures
we have information here not available in the access log, especially
if the /api2/extjs formatter is used, which encapsulates errors in a
200 response.
So keep the auth log for now, but extend it use from create ticket
calls to all authentication failures for API calls, this ensures one
can also fail2ban tokens.
Do that logging in a central place, which makes it simple but means
that we do not have the user ID information available to include in
the log.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 15:44:07 +0000 (16:44 +0100)]
worker task: fix passing upid to send command
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 16:15:11 +0000 (17:15 +0100)]
log rotate: only add .zst to new file after second rotation
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Wed, 4 Nov 2020 13:20:44 +0000 (14:20 +0100)]
ui: fix task description for log rotate
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Dietmar Maurer [Wed, 4 Nov 2020 10:27:57 +0000 (11:27 +0100)]
proxy: use new datastore notify settings
Dietmar Maurer [Wed, 4 Nov 2020 09:42:22 +0000 (10:42 +0100)]
config: allow to configure who receives job notify emails
Dominik Csapak [Wed, 4 Nov 2020 10:34:14 +0000 (11:34 +0100)]
ui: SyncJobEdit: fix sending 'delete' values on SyncJob creation
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dietmar Maurer [Wed, 4 Nov 2020 08:10:56 +0000 (09:10 +0100)]
sync: improve log format
Dietmar Maurer [Wed, 4 Nov 2020 07:49:35 +0000 (08:49 +0100)]
proxy: log if there are too many open connections
Dietmar Maurer [Wed, 4 Nov 2020 07:20:36 +0000 (08:20 +0100)]
fix bug #2870: use updated tickets
Dietmar Maurer [Wed, 4 Nov 2020 06:12:55 +0000 (07:12 +0100)]
ui: set default deduplication factor to 1.0
Dietmar Maurer [Wed, 4 Nov 2020 05:52:10 +0000 (06:52 +0100)]
proxy: use env PROXMOX_DEBUG to enable/disable debug output
We only print early connection errors when this env var is set.
Dietmar Maurer [Wed, 4 Nov 2020 05:35:44 +0000 (06:35 +0100)]
cleanup: use Arc to count open connections
Dietmar Maurer [Tue, 3 Nov 2020 12:14:10 +0000 (13:14 +0100)]
fix #3106: correctly queue incoming connections
Thomas Lamprecht [Tue, 3 Nov 2020 16:41:26 +0000 (17:41 +0100)]
bump version to 0.9.5-1
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Tue, 3 Nov 2020 16:29:17 +0000 (17:29 +0100)]
report: add more commands/files to check
add all of our configuration files in /etc/proxmox-backup/ further,
call some ZFS tool to get that status.
Also, use the subscription command form manager, as we often require
more info than the status. Also, adapt formatting a bit.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Tue, 3 Nov 2020 16:27:12 +0000 (17:27 +0100)]
report: avoid lazy_static for command/files/.. definitions
those are not in a hot code path, and it is not really much work to
build them on the go..
It may not matther much, but it is unnecessary. Rust will probably
inline most of it anyway..
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Tue, 3 Nov 2020 16:23:43 +0000 (17:23 +0100)]
manager: report: call method directly, avoid HTTPS request
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Hannes Laimer [Tue, 3 Nov 2020 12:29:08 +0000 (13:29 +0100)]
manager: add report cli command
Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
Hannes Laimer [Tue, 3 Nov 2020 12:29:07 +0000 (13:29 +0100)]
report: add webui button for system report
Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
Hannes Laimer [Tue, 3 Nov 2020 12:29:06 +0000 (13:29 +0100)]
report: add api endpoint and function to generate report
Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
Thomas Lamprecht [Tue, 3 Nov 2020 13:49:04 +0000 (14:49 +0100)]
ui: task filter: add button icons
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Tue, 3 Nov 2020 13:47:59 +0000 (14:47 +0100)]
ui: task filter: reorder to avoid wasting vertical space
Includes some eslint fixes and label changes as well, was to much
work to split that out in its own commit.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Dominik Csapak [Fri, 30 Oct 2020 14:02:15 +0000 (15:02 +0100)]
ui: add panel/Tasks and use it for the node tasks
this is a panel that is heavily inspired from widget-toolkits
node/Tasks panel, but is adapted to use the extended api calls of
pbs (e.g. since/until filter)
has 'filter' panel (like pmgs log tracker gui), but it is collapsible
if we extend the api calls of the other projects, we can merge this
again into the widget-toolkit one and use that
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Fri, 30 Oct 2020 14:02:14 +0000 (15:02 +0100)]
api2/node/tasks: add optional until filter
so that users select specific time ranges with 'since' and 'until'
(e.g. a single day)
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Fri, 30 Oct 2020 14:02:13 +0000 (15:02 +0100)]
api2/status: remove list_task api call
we do not need it anymore, we can do everything with nodes/NODE/tasks
instead
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Fri, 30 Oct 2020 14:02:12 +0000 (15:02 +0100)]
api2/node/tasks: add optional since/typefilter/statusfilter
and change all users of the /status/tasks api call to this
with this change we can now delete the /status/tasks api call
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Fri, 30 Oct 2020 14:02:11 +0000 (15:02 +0100)]
api2/node/tasks: change limit behaviour when it is 0
instead of returning 0 elements (which does not really make sense anyway),
change it so that there is no limit anymore (besides usize::MAX)
this is technically a breaking change for the api, but i guess
no one is using limit=0 for anything sensible anyway
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Fri, 30 Oct 2020 14:02:10 +0000 (15:02 +0100)]
server/worker_task: add tasktype to return the api type of a taskstate
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dietmar Maurer [Tue, 3 Nov 2020 05:36:50 +0000 (06:36 +0100)]
cleanup: use const_regex, use BACKUP_ID_REGEX for api too
Dominik Csapak [Mon, 2 Nov 2020 12:08:04 +0000 (13:08 +0100)]
fix #3039: use the same ID regex for info and api
in the api we use PROXMOX_SAFE_ID_REGEX for backup ids, but here
(where we use it to list them) we use a local regex
since the first is a superset of the one used here, simply extend
the local one
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Fabian Grünbichler [Mon, 2 Nov 2020 10:48:11 +0000 (11:48 +0100)]
docs: extend managing remotes
with information about required privileges and limitations
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Mon, 2 Nov 2020 10:48:10 +0000 (11:48 +0100)]
sync: add access check tests
should cover all the current scenarios. remote server-side checks can't
be meaningfully unit-tested, but they are simple enough so should
hopefully never break.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Mon, 2 Nov 2020 10:48:09 +0000 (11:48 +0100)]
user.cfg/user info: add test constructors
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 19:56:30 +0000 (20:56 +0100)]
proxy: drop now unused UPID import
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Dominik Csapak [Mon, 2 Nov 2020 11:34:39 +0000 (12:34 +0100)]
proxmox-backup-proxy: use only jobstate for garbage_collection schedule
in case the garbage_collection errors out, we never set the in-memory
state, so if it failed, the last 'good' starttime was considered
for the schedule
this could lead to the job running every minute instead of the
correct schedule
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Mon, 2 Nov 2020 11:34:38 +0000 (12:34 +0100)]
api2/admin/datastore: start the garbage_collection task with our helper
instead of manually, this has the advantage that we now set
the jobstate correctly and can return with an error if it is
currently running (instead of failing in the task)
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Mon, 2 Nov 2020 11:34:37 +0000 (12:34 +0100)]
server/gc_job: add 'to_stdout'
we will use this for the manual api call
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Dominik Csapak [Mon, 2 Nov 2020 11:34:36 +0000 (12:34 +0100)]
backup/{dynamic, fixed}_index: improve error message for small index files
index files that were smaller than their respective header size,
would fail with
"failed to fill whole buffer"
instead now check explicitely for the size and fail with
"index too small (size)"
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 19:50:24 +0000 (20:50 +0100)]
gc: mark_used_chunks: reduce implementation noise
try do reduce some unecessary lines, make match arms more precise so
one can faster see what's actually happening.
Also, avoid
> return Err(format_err!(...))
stuff, just use bail!()
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Dominik Csapak [Mon, 2 Nov 2020 11:34:35 +0000 (12:34 +0100)]
garbage collect: improve index error messages
so that in case of a broken index file, the user knows which it is
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 18:31:56 +0000 (19:31 +0100)]
log rotate task: make task archive limits be binary based
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 18:30:49 +0000 (19:30 +0100)]
log rotate task: adapt internal jobstate ID, set worker one to None for now
as we have only one logrotate task currently..
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 18:21:58 +0000 (19:21 +0100)]
server: implement access log rotation with re-open via command socket
re-use the future we already have for task log rotation to trigger
it.
Move the FileLogger in ApiConfig into an Arc, so that we can actually
update it and REST using the new one.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 18:18:36 +0000 (19:18 +0100)]
server: write main daemon PID to run directory
so that we can easily get the main PID of the last recently launched
daemon. Will be used to get the control socket of that one for access
lgo rotate in a future patch
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 18:23:18 +0000 (19:23 +0100)]
command socket: make create_control_socket private
this is internal for now, use the comanndo socket struct
implementation, and ideally not a new one but the existing ones
created in the proxy and api daemons.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 18:13:36 +0000 (19:13 +0100)]
server: use generalized commando socket for worker tasks commands
Allows to extend the use of that socket in the future, e.g., for log
rotate re-open signaling.
To reflect this we use a more general name, and change the commandos
to a more clear namespace.
Both are actually somewhat a breaking change, but the single real
world issue it should be able to cause is, that one won't be able to
stop task from older daemons, which still use the older abstract
socket name format.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 09:12:56 +0000 (10:12 +0100)]
server: add CommandoSocket where multiple users can register commands
This is a preparatory step to replace the task control socket with it
and provide a "reopen log file" command for the rest server.
Kept it simple by disallowing to register new commands after the
socket gets spawned, this avoids the need for locking.
If we really need that we can always wrap it in a Arc<RWLock<..>> or
something like that, or even nicer, register at compile time.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 17:52:47 +0000 (18:52 +0100)]
config: network: use error message when parsing netmask failed
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 18:26:59 +0000 (19:26 +0100)]
tools: file logger: avoid some possible unwraps in log method
writing to a file can explode quite easily.
time formatting to rfc3339 should be more robust, but it has a few
conditions where it could fail, so catch that too (and only really
do it if required).
The writes to stdout are left as is, it normally is redirected to
journal which is in memory, and thus breaks later than most stuff,
and at that point we probably do not care anymore anyway.
It could make sense to actually return a result here..
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 17:31:31 +0000 (18:31 +0100)]
tools: log rotate: compressing rotated files
We renamed the last one always to a file without compression
extension, even if it was .zst previously. So always add the correct
ending to the new last one, if compress was true.
Further, we cannot detect if there'd be a compression required if we
rotated (renamed) it already to the file with .zst included.
So check on rotation itself if it would be a "no .zst" -> ",zst"
transition, and call compress there.
it really should be OK now *knocking wood*
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 16:45:02 +0000 (17:45 +0100)]
tools: log rotate: fix file ending for compressed files
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Dietmar Maurer [Mon, 2 Nov 2020 13:32:35 +0000 (14:32 +0100)]
config: support netmask when parsing interfaces file
Dietmar Maurer [Mon, 2 Nov 2020 09:03:36 +0000 (10:03 +0100)]
ui: fix verificationjob task description
Thomas Lamprecht [Mon, 2 Nov 2020 07:51:24 +0000 (08:51 +0100)]
file logger: allow reopening file
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Mon, 2 Nov 2020 07:50:56 +0000 (08:50 +0100)]
pull out /run and /var/log directory constants to buildcfg
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Dietmar Maurer [Mon, 2 Nov 2020 08:11:08 +0000 (09:11 +0100)]
notify: add link to server GUI
Dietmar Maurer [Mon, 2 Nov 2020 07:02:51 +0000 (08:02 +0100)]
gc: remove extra empty lines in email notification template
Thomas Lamprecht [Mon, 2 Nov 2020 07:08:25 +0000 (08:08 +0100)]
ui: fix dashboard subscription
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Fabian Grünbichler [Fri, 30 Oct 2020 11:36:42 +0000 (12:36 +0100)]
sync: allow sync for non-superusers
by requiring
- Datastore.Backup permission for target datastore
- Remote.Read permission for source remote/datastore
- Datastore.Prune if vanished snapshots should be removed
- Datastore.Modify if another user should own the freshly synced
snapshots
reading a sync job entry only requires knowing about both the source
remote and the target datastore.
note that this does not affect the Authid used to authenticate with the
remote, which of course also needs permissions to access the source
datastore.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Fabian Grünbichler [Fri, 30 Oct 2020 11:36:41 +0000 (12:36 +0100)]
fix #2864: add owner option to sync
instead of hard-coding 'backup@pam'. this allows a bit more flexibility
(e.g., syncing to a datastore that can directly be used as restore
source) without overly complicating things.
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Thomas Lamprecht [Sun, 1 Nov 2020 12:19:29 +0000 (13:19 +0100)]
manager: subscription commands s/delete/remove/
no idea why I added it as "delete", for all other such operations we
use the "remove" sub-command...
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>