Adam Grare

Okay @Dhamo1107 can you try testing out the latest updates in this PR, un-comment the collection in the openstack persister if you commented it out, and try applying the changes...

@miq-bot cross-repo-tests /all, https://github.com/ManageIQ/manageiq-gems-pending/pull/623

This looks great @putmanoj safe to say you can take it out of Draft and let specs run I think unless you are waiting on something else.

The mixture of strings and symbols looks weird and pretty fragile, but it looks like that is the way the existing code is as well and is required to be...

@miq-bot cross-repo-tests https://github.com/ManageIQ/manageiq-providers-dummy_provider/pull/77

Down to 5 failures: ``` rspec ./spec/models/manageiq/providers/cloud_manager/provision_workflow_spec.rb:186 # ManageIQ::Providers::CloudManager::ProvisionWorkflow with VPC relationships #allowed_cloud_networks without a zone rspec ./spec/models/manageiq/providers/inventory/persister/serializing_spec.rb:17 # ManageIQ::Providers::Inventory::Persister tests we can serialize inventory object with nested lazy references...

So while we don't want to prevent deletion at the model level, we do want to prevent the user from deleting or editing these via the UI/API. The API has...

> ReadOnlyMixin does want to prevent deletion at the model level, which is why it has a special bypass for seeding. That is, read_only generally means "seeded", and so that...

> Can any of our 🐧 friends require qpid_proton? ``` vmdb(dev)> require 'qpid_proton' /home/grare/adam/.gem/ruby/3.3.0/gems/qpid_proton-0.37.0/lib/cproton.so: warning: undefining the allocator of T_DATA class swig_runtime_data => true ``` This was on master, let...

`qpid_proton` libraries and the gem should be installed by `bin/before_install` in both openstack and nuage, https://github.com/ManageIQ/manageiq-providers-openstack/blob/master/bin/before_install#L6-L10 This `bin/before_install` script should be run by cross-repo tests