[PP-1] AM:A Drush command should execute migrations as user 1, just like the UI

The d7_file_private migration works fine for the initial file import

👍

however the d7_file_entity:{default, image, video, audio}:private migrations fails for every file, with the error "You do not have access to the referenced entity".

This is possibly a similar issue to #3134245: Run drush imports as user=1 in migrate_tools.

Hm … interesting! That seems like a very plausible theory 🤔

Just to make sure: could you please run the "Private files" migration through the AM:A UI and let me know whether that works correctly? 🙏

(The AM:A UI is always used by user 1 so this should be the case. But I'd like to confirm that before digging in and working on this.)

Thanks!

Hi Wim, I can confirm that the private files migration works correctly through the AM:A UI, it's only when using drush ama:import that it fails with this error.

Thanks!

👍 Thanks, that was very fast!

I am swamped this month, so I doubt I'd be able to get to this this month. Maybe in January. If you are willing to create an MR against src/Commands/AcquiaMigrateCommands.php, it could go a lot faster 😇😄

Thanks for the merge request, @jienckebd! 🥳

The logic changes look good — just some minor remarks and this'll be good to go 😊

Investigating the first random failure in 📌 CI: frequent (random?) failures of InlineEntityFormCompatibilityTest in "🏚️→ 🏠 Contrib migrations Drupal 10 (next)" job Active . This issue is clearly blocked by that 😬

Updated MR to support both "drush ama:*" and "drush migrate:*" commands.

Why are we modifying a drush built-in command in this MR now? 😱 And that even without linking to the corresponding upstream issue?! (I spot no @see)

How do we ensure that this won't break when it is fixed upstream?