Skip to content

Conversation

@sathish-progress
Copy link
Contributor

@sathish-progress sathish-progress commented May 17, 2023

Note: inspec-azure will continue to work. Using inspec exec -t azure:// will continue to work.

Warns the users that long unused Azure connections from train would be deprecated.

Description

As part of removing the long unused Azure SDKs from train, this change is the first part where we let know the users about deprecating this Azure Connection.
While target platform azure meaning -t azure would still continue to work.
Things that will be deprecated -

  • msi authentication
  • any azure SDK based connections including graph rbac, vault, storage. These resources are long supported in inspec-azure cloud resource packs through non-SDK means using latest Azure APIs and they will continue to exist.
Screenshot 2023-07-25 at 5 11 54 PM

Related Issue

second part: #745
https://chefio.atlassian.net/browse/CHEF-2414

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New content (non-breaking change)
  • Breaking change (a content change which would break existing functionality or processes)

Checklist:

  • I have read the CONTRIBUTING document.

@sathish-progress sathish-progress force-pushed the feature/CHEF-2414-warn-azure-deprecation branch from ecf1330 to 0f4fa49 Compare May 17, 2023 08:53
Signed-off-by: Sathish <[email protected]>
Copy link
Contributor

@clintoncwolfe clintoncwolfe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to ask that we do this without using ActiveSupport. We try to keep it out of train-core (and no, azure transport is not in train-core) but I think the only reason train has an active_support dependency was due to azure. I think active support dependencies need to be well-justified. Here we could get by with a warning.

@clintoncwolfe clintoncwolfe merged commit cff6334 into main Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants