[pve-devel] [PATCH container] remove unused PVE::HA::Env::PVE2 use clause

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Nov 15 09:19:34 CET 2016


Ok, we need this for registering the HA resources, so this patches are 
wrong!

But we should not need it like it is currently, I'm looking if I find 
another way to remove the cyclic dependency...

On 11/15/2016 09:10 AM, Thomas Lamprecht wrote:
> This also removes a cyclic dependency, which looked like:
> PVE::HA::Env::PVE2 -> PVE::HA::Resource::PVECT -> PVE::API2::LXC ->
> PVE::HA::Env::PVE2
>
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> ---
>   src/PVE/API2/LXC.pm        | 1 -
>   src/PVE/API2/LXC/Status.pm | 1 -
>   2 files changed, 2 deletions(-)
>
> diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm
> index 38b1feb..2e0e3b1 100644
> --- a/src/PVE/API2/LXC.pm
> +++ b/src/PVE/API2/LXC.pm
> @@ -19,7 +19,6 @@ use PVE::LXC::Migrate;
>   use PVE::API2::LXC::Config;
>   use PVE::API2::LXC::Status;
>   use PVE::API2::LXC::Snapshot;
> -use PVE::HA::Env::PVE2;
>   use PVE::HA::Config;
>   use PVE::JSONSchema qw(get_standard_option);
>   use base qw(PVE::RESTHandler);
> diff --git a/src/PVE/API2/LXC/Status.pm b/src/PVE/API2/LXC/Status.pm
> index 83a23f5..ee1bb4f 100644
> --- a/src/PVE/API2/LXC/Status.pm
> +++ b/src/PVE/API2/LXC/Status.pm
> @@ -15,7 +15,6 @@ use PVE::RESTHandler;
>   use PVE::RPCEnvironment;
>   use PVE::LXC;
>   use PVE::LXC::Create;
> -use PVE::HA::Env::PVE2;
>   use PVE::HA::Config;
>   use PVE::JSONSchema qw(get_standard_option);
>   use base qw(PVE::RESTHandler);





More information about the pve-devel mailing list