Jason Frey
Jason Frey
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 bypass...
> Also the seeded `Classifications` aren't read_only, https://github.com/ManageIQ/manageiq/blob/master/db/fixtures/classifications.yml#L5 so we aren't preventing seeded classifications from being deleted today as it is. Right I'm saying they should have been (more bugs)....
Reopened version of #23651 cc @agrare --- Must pass cross-repo tests before merge @miq-bot cross-repo-tests manageiq-providers-nuage manageiq-providers-openstack
This looks like a duplicate of https://github.com/ManageIQ/manageiq-ui-classic/issues/9555 which was fixed in https://github.com/ManageIQ/manageiq-ui-classic/pull/9392
Openstack failures look like they might be related.
Actually maybe not? I can't require "qpid_proton" locally even before this change (but maybe I don't have the right libs installed)
I saw a technique in another repository recently where they used multiple jobs. In that repo, the primary job would test only on ubuntu-latest, and then if that succeeded, a...
Thanks for the report! @elsamaryv Please take a look at this one.
I don't follow - I removed the newline on purpose in https://github.com/ManageIQ/manageiq/pull/23215/files#diff-0b1a6bddeecb64b558cf43f0da0280d86b91d58f9b0dd89e0e3cf52b136d80b4R1512
> It looks more readable to me... what do you think? Honestly it's harder to read for me :)