Fix create/update deployment from file not respecting astro executor for standard deployments by feluelle · Pull Request #1892 · astronomer/astro-cli
Description
Describe the purpose of this pull request.
Currently, when trying to create a standard deployment from a file using Astro executor, we get:
Error: Missing fields: executor
because it is not being added to the request payload. This PR adds that field to standard deployment creation and update requests.
🎟 Issue(s)
Related #XXX
🧪 Functional Testing
List the functional testing steps to confirm this feature or fix.
📸 Screenshots
Add screenshots to illustrate the validity of these changes.
📋 Checklist
- Rebased from the main (or release if patching) branch (before testing)
- Ran
make testbefore taking out of draft - Ran
make lintbefore taking out of draft - Added/updated applicable tests
- Tested against Astro-API (if necessary).
- Tested against Houston-API and Astronomer (if necessary).
- Communicated to/tagged owners of respective clients potentially impacted by these changes.
- Updated any related documentation