-
Notifications
You must be signed in to change notification settings - Fork 131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release-0.47] Bump upjet to 1.1.3 #1195
[release-0.47] Bump upjet to 1.1.3 #1195
Conversation
/test-examples="examples/ec2/vpc.yaml" |
/test-examples="examples/ec2/vpc.yaml" |
f4b0fc0
to
4a664f7
Compare
/test-examples="examples/ec2/vpc.yaml" |
Because of the shared runner disk issues, for now, we ran the following jobs locally:
Also, the uptest runs cover the local-deploy job. In other words, in uptest runs, we ensure that the packages are correctly deployed because the work done in |
Regarding the failing jobs, let me open a PR which conditionally introduces the disk cleanup step and see if it helps here. I remember these were failing previously for this branch. |
Signed-off-by: Sergen Yalçın <[email protected]>
- apigateway.IntegrationResponse - rds.v1beta1.Instance Signed-off-by: Sergen Yalçın <[email protected]>
- docdb.ClusterInstance - elasticache.Cluster - elasticache.ReplicationGroup Signed-off-by: Sergen Yalçın <[email protected]>
Signed-off-by: Sergen Yalçın <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sergenyalcin. Please see #1188 for the review.
Signed-off-by: Sergen Yalçın <[email protected]>
Description of your changes
This PR bumps the upjet dependency to 1.1.2 for including StateFunc fix.
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested