[pbs-devel] [PATCH proxmox-backup] unbreak get_subscription without sshd installed

Fabian Grünbichler f.gruenbichler at proxmox.com
Tue Nov 10 17:52:11 CET 2020


> Dietmar Maurer <dietmar at proxmox.com> hat am 10.11.2020 17:01 geschrieben:
> 
>  
> Or make sure sshd is installed?

we don't require it for any functionality (except deriving the server ID, which can be done with any arbitrary somewhat static, server-specific file)

> > On 11/10/2020 4:59 PM Fabian Grünbichler <f.gruenbichler at proxmox.com> wrote:
> > 
> >  
> > which might be the case e.g. in a PBS container
> > 
> > Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
> > ---
> > 
> > Notes:
> >     we might want to switch to a different file as base? or even generate a file
> >     just for the server ID derivation?
> > 
> >  src/api2/node/subscription.rs | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/api2/node/subscription.rs b/src/api2/node/subscription.rs
> > index 67797fd5..010a0209 100644
> > --- a/src/api2/node/subscription.rs
> > +++ b/src/api2/node/subscription.rs
> > @@ -94,7 +94,7 @@ pub fn get_subscription(
> >          Ok(None) => SubscriptionInfo {
> >              status: SubscriptionStatus::NOTFOUND,
> >              message: Some("There is no subscription key".into()),
> > -            serverid: Some(tools::get_hardware_address()?),
> > +            serverid: tools::get_hardware_address().ok(),
> >              url:  Some(url.into()),
> >              ..Default::default()
> >          },
> > -- 
> > 2.20.1
> > 
> > 
> > 
> > _______________________________________________
> > pbs-devel mailing list
> > pbs-devel at lists.proxmox.com
> > https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel





More information about the pbs-devel mailing list