cloudian: Set cloudian.connector.enabled as not dynamic

[davidjumani]

Description

This PR sets cloudian.connector.enabled as not dynamic since cloudian requires the cloudianSsoLogin API which is added only once the MS is restarted

Types of changes

• Breaking change (fix or feature that would cause existing functionality to change)
• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)

Bug Severity

• BLOCKER
• Critical
• Major
• Minor
• Trivial

[davidjumani]

@blueorangutan package

[blueorangutan]

@davidjumani a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[blueorangutan]

Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 435

[davidjumani]

@blueorangutan test

[blueorangutan]

@davidjumani a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[sureshanaparti]

code LGTM

[blueorangutan]

Trillian test result (tid-483)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 32027 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4938-t483-kvm-centos7.zip
Smoke tests completed. 87 look OK, 0 have error(s)
Only failed tests results shown below:

Test	Result	Time (s)	Test File

[yadvr]

@davidjumani this won't work - maybe for UI but for the plugin to instantiate - you'll need to restart mgmt server, as it's used in the configure method of manager/class

[davidjumani]

Exactly why it's being set as non dynamic. The user will get a notification that the MS needs to be restarted after this change and on restart it'll pick up the necessary API

[yadvr]

Ah I see, my bad I misunderstood - lgtm.