Results 247 comments of Adam Grare

`azure_mgmt_*` are used by the azure_stack provider, and has been discontinued by microsoft unfortunately https://github.com/Azure/azure-sdk-for-ruby#important-announcement so I doubt we'll be able to effect any major changes there.

> It is no longer supported and they have an alternate interface. Moving to the new one should be a win. no? Are you talking about `MsRestAzure`? This is not...

@kbrock `Switch` is just a base class for both `HostVirtualSwitch` and `DistributedVirtualSwitch` so I don't think we should say that `::Switch` is a `HostVirtualSwitch` Having a switch associate to the...

> Do we need to clean these up in another way? Like deleting all files underneath that directory (may cause issues for local development though) Yeah I thought I'd have...

Okay it looks like leftover locks are from these two spec files (not surprisingly), `spec/models/git_repository_spec.rb` and `spec/models/manageiq/providers/embedded_ansible/automation_manager/configuration_script_source_spec.rb`.

Does anything ever delete these git_repository lockfiles at runtime?

WIP until I cover the other specs that leave these lockfiles around

After running the full suite no lockfiles are left over: ``` $ rake parallel:spec ... Randomized with seed 19268 12171 examples, 0 failures, 4 pendings Took 46 seconds $ ls...

@kbrock fyi another fix_auth sporadic failure ``` 1) FixAuth::AuthModel#miq_database uses random numbers for invalid Failure/Error: expect(ManageIQ::Password.decrypt(bad.session_secret_token)).to_not eq "newpass" ManageIQ::Password::PasswordError: cannot decrypt encrypted string # ./spec/tools/fix_auth/auth_model_spec.rb:118:in `block (3 levels) in '...