Skip to content
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

Adds controller tag support to instance profile. #13352

Merged
merged 1 commit into from Sep 24, 2021

Conversation

tlm
Copy link
Member

@tlm tlm commented Sep 23, 2021

Instance profiles created by the controller now contain the controller
tag that they are made for.

Checklist

  • Requires a pylibjuju change
  • Added integration tests for the PR
  • Added or updated doc.go related to packages changed
  • Comments answer the question of why design decisions were made

QA steps

Bootstrap an AWS controller with juju bootstrap aws/ap-southeast-2 --bootstrap-constraints="instance-role=auto" testmctestface

Check that the corresponding Instance Profile has the controller uuid tag set with:

aws iam list-instance-profile-tags --instance-profile-name "juju-controller-testmctestface"

Instance profiles created by the controller now contain the controller
tag that they are made for.
@hpidcock hpidcock added the 2.9 label Sep 23, 2021
Copy link
Member

@hpidcock hpidcock left a comment

Choose a reason for hiding this comment

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

seems straightforward LGTM

@tlm
Copy link
Member Author

tlm commented Sep 23, 2021

$$merge$$

1 similar comment
@tlm
Copy link
Member Author

tlm commented Sep 24, 2021

$$merge$$

@jujubot jujubot merged commit e9248ef into juju:2.9 Sep 24, 2021
jujubot added a commit that referenced this pull request Sep 28, 2021
#13361

Merge from 2.9 to bring forward:
- #13360 from wallyworld/simplestreams-compression
- #13359 from manadart/2.9-lxd-container-images
- #13352 from tlm/aws-instance-profile
- #13358 from jujubot/increment-to-2.9.16
- #13354 from wallyworld/refresh-consume-proxy
- #13353 from wallyworld/cmr-consume-fixes
- #13346 from SimonRichardson/raft-api-client
- #13349 from wallyworld/remove-orphaned-cmrapps
- #13348 from benhoyt/fix-secretrotate-tests
- #13119 from SimonRichardson/pass-context
- #13342 from SimonRichardson/raft-facade
- #13341 from ycliuhw/feature/quay.io

Conflicts (easy resolution):
- apiserver/common/crossmodel/interface.go
- apiserver/errors/errors.go
- apiserver/params/apierror.go
- apiserver/testserver/server.go
- scripts/win-installer/setup.iss
- snap/snapcraft.yaml
- version/version.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants