Skip to content

Conversation

@hankehly
Copy link
Contributor

@hankehly hankehly commented Aug 27, 2022

Related: #24099 #25952

Summary

This PR sets the template_fields property on RdsCreateDbInstanceOperator and RdsDeleteDbInstanceOperator to allow users to programmatically change the database id, instance size, allocated storage, etc..

It also replaces use of @task decorated functions with their appropriate operator in system tests.

Todo

  • Add template_fields to RDS operators
  • Update/run system tests



@task
def create_rds_instance(db_name, instance_name) -> None:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made the same updates in each system test file

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

@hankehly
Copy link
Contributor Author

@o-nikolas @vincbeck @ferruzzi @potiuk
Please review this at your earliest convenience.

@hankehly hankehly requested a review from kaxil as a code owner August 28, 2022 03:54
Co-authored-by: D. Ferruzzi <ferruzzi@amazon.com>
@hankehly hankehly requested review from ferruzzi and removed request for kaxil August 30, 2022 00:18
@ferruzzi
Copy link
Contributor

Cool, thanks for the change. LGTM.

@hankehly
Copy link
Contributor Author

hankehly commented Sep 1, 2022

@o-nikolas @vincbeck @ferruzzi @potiuk
Could someone please merge this PR?

@kaxil kaxil merged commit 751cb9a into apache:main Sep 5, 2022
@hankehly hankehly deleted the add_template_fields_to_rds_operators branch September 6, 2022 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants