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

Amazon Connect #16392

Closed
abebars opened this issue Nov 23, 2020 · 29 comments
Closed

Amazon Connect #16392

abebars opened this issue Nov 23, 2020 · 29 comments
Labels
new-data-source Introduces a new data source. new-resource Introduces a new resource. service/connect Issues and PRs that pertain to the connect service.

Comments

@abebars
Copy link
Contributor

abebars commented Nov 23, 2020

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

Amazon Connect is omnichannel cloud contact center service that allows building a contact center across voice & chat channels.

New or Affected Resource(s)

  • aws_connect_insntace

Potential Terraform Configuration

resource "aws_connect_insntace" "example" {
  name                     = "example-connect-instance"
  directory_id             = "d-12345678"
  identity_management_type = "self_managed"
  alias                    = "example-alias"
  inbound_call_enabled     = true
  outbound_call_enabled    = true
}

References

Developer Guide.

Requires AWS SDK v1.35.33

@abebars abebars added the enhancement Requests to existing resources that expand the functionality or scope. label Nov 23, 2020
@github-actions github-actions bot added the needs-triage Waiting for first response or review from a maintainer. label Nov 23, 2020
@ewbankkit ewbankkit added new-resource Introduces a new resource. service/connect Issues and PRs that pertain to the connect service. labels Nov 23, 2020
@bflad bflad removed enhancement Requests to existing resources that expand the functionality or scope. needs-triage Waiting for first response or review from a maintainer. labels Dec 1, 2020
@thornleyk
Copy link
Contributor

thornleyk commented Dec 5, 2020

I am actually building this at the moment, I am wondering if there has been some work on this already as when I was adding the required hooks I saw that connect was already listed in the labels-servcie.tf I don't really want to duplicate effort if this is already underway

@abebars
Copy link
Contributor Author

abebars commented Dec 5, 2020

I am actually building this at the moment, I am wondering if there has been some work on this already as when I was adding the required hooks I saw that connect was already listed in the labels-servcie.tf I don't really want to duplicate effort if this is already underway

Thanks for the update on that. I am currently working on this as well. I am not aware of any other work under the way for this but I may be mistaken. I should have something ready in the next couple weeks.

@thornleyk
Copy link
Contributor

I am actually building this at the moment, I am wondering if there has been some work on this already as when I was adding the required hooks I saw that connect was already listed in the labels-servcie.tf I don't really want to duplicate effort if this is already underway

Thanks for the update on that. I am currently working on this as well. I am not aware of any other work under the way for this but I may be mistaken. I should have something ready in the next couple weeks.

Need some help?

@abebars
Copy link
Contributor Author

abebars commented Dec 6, 2020

I am actually building this at the moment, I am wondering if there has been some work on this already as when I was adding the required hooks I saw that connect was already listed in the labels-servcie.tf I don't really want to duplicate effort if this is already underway

Thanks for the update on that. I am currently working on this as well. I am not aware of any other work under the way for this but I may be mistaken. I should have something ready in the next couple weeks.

Need some help?

I can share the branch early with you in the next few days for early checks or we can collaborate together if you have something in mind?

@thornleyk
Copy link
Contributor

I am actually building this at the moment, I am wondering if there has been some work on this already as when I was adding the required hooks I saw that connect was already listed in the labels-servcie.tf I don't really want to duplicate effort if this is already underway

Thanks for the update on that. I am currently working on this as well. I am not aware of any other work under the way for this but I may be mistaken. I should have something ready in the next couple weeks.

Need some help?

I can share the branch early with you in the next few days for early checks or we can collaborate together if you have something in mind?

collaboration would be good, not sure what the scope of the target terraform resources you had in mind but I need to get the complete connect resource graph working with external dependencies. Currently planning:

instance and associated lambda and lex associations
contact_flow and the relationships to lex and lambda alias's
users and roles

I have the start of the contact flow resource and data source, I only started yesterday, took a while to understand the provider architecture, potentially we can package up the work and eat the elephant bit by bit here is my current branch if your interested https://github.com/thornleyk/terraform-provider-aws/tree/feature/add-connect-assets Its super rough at the moment have been using the lex resource provider as a guide to implementation as it features similar resource relationships

@abebars
Copy link
Contributor Author

abebars commented Dec 6, 2020

@thornleyk That looks good. I started from the instance and it's config so we are not overlapping so far 😅. I was thinking the same about tackling this project piecemeal. So we can have multiple PRs and also others can work in other sub resources if any one is interested 😁. So will keep you posted on my progress so we can get it sooner in working state for all connect resources.

@thornleyk
Copy link
Contributor

thornleyk commented Dec 6, 2020

@thornleyk That looks good. I started from the instance and it's config so we are not overlapping so far 😅. I was thinking the same about tackling this project piecemeal. So we can have multiple PRs and also others can work in other sub resources if any one is interested 😁. So will keep you posted on my progress so we can get it sooner in working state for all connect resources.

That sounds great, we are building new so my main focus in on contact flows and using a static instance at the moment so I think we are good from an overlap perspective, I think we have a few hurdles to get over with some of the lack of functionality in the SDK e.g. we can't create or delete an instance which makes ACC tests hard, I also have the same trouble in contact flows, I can't delete so am just renaming to ZZ_Trash. These factors to really impact ACC regression runs as the resources keep stacking up, not so bad once the resource is complete but its a pain when you are starting out as the iterations are high.

I think one thing perhaps we can start on is what the resource graph looks like e.g. how does instance and contact flow relate, what are the dependencies and what is exchanged, also naming conventions e.g. instance_id etc. I have DM'ed you linked in if you want to chat via your preferred messaging platform

Based on your presentation at invent it also looks like we have the same struggles :)

@thornleyk
Copy link
Contributor

@abebars reading the contribution guide we should be submitting the new service and resources in stages to minimise pull request review time, reading this we should submit the service first with no resources and then follow this one resource at a time. I suggest that we do service first then your connect instance resource, then I can leverage the instance id generated by your resource in my connect flow resource. thoughts?

@abebars
Copy link
Contributor Author

abebars commented Dec 9, 2020

@abebars reading the contribution guide we should be submitting the new service and resources in stages to minimize pull request review time, reading this we should submit the service first with no resources and then follow this one resource at a time. I suggest that we do service first then your connect instance resource, then I can leverage the instance id generated by your resource in my connect flow resource. thoughts?

Sounds like a plan to me. I will get the instance PR ready soon and then will work on the config association in different PRs

@zach-anthony
Copy link
Contributor

@abebars @thornleyk keen to collaborate as well and help out on any parts that you haven't started or provide any input for reviews. Just let me know what would be best to start on so we don't overlap

@thornleyk
Copy link
Contributor

@abebars @thornleyk keen to collaborate as well and help out on any parts that you haven't started or provide any input for reviews. Just let me know what would be best to start on so we don't overlap

Cheers all input welcome! I'm staring from contact flow back to instance and I think @abebars is working from instance to contact flow.

I am intending on setting up some basic instance data sources based off alias so that I can hold my resource up with variable instance attributes

From there I'm looking at linking in Lex and Lambda aliases from the core instance

One thing I could help with is some idea of the full resource graph and attributes, happy to collaborate on this but would love some 3rd party input

@thornleyk
Copy link
Contributor

Here is my current branch for the connect flow resource https://github.com/thornleyk/terraform-provider-aws/tree/f-aws_connect_contact_flow

I also have a temp branch for the connect instance resource, where the data source is going to be a dependency to the connect flow https://github.com/thornleyk/terraform-provider-aws/tree/f-aws_connect_instance

@abebars
Copy link
Contributor Author

abebars commented Dec 11, 2020

@thornleyk just drafted this PR #16709 , let me know wdyt

@abebars
Copy link
Contributor Author

abebars commented Dec 16, 2020

@thornleyk #16644 is merged and #16709 is ready. Let me know if I am missing anything. I will branch off my or to work on the other association for the instance like storage , origins , ... . I should have a PR for this by the end of the week.

@thornleyk
Copy link
Contributor

@abebars Hey I have just created the WIP PR for Contact Flows, once your branch is merged this commit will become smaller. Would be good if you could have a review, not sure how to add reviewers in github #16854

@thornleyk
Copy link
Contributor

thornleyk commented Feb 5, 2021

@bflad would it be possible to help us move these resources along, please :) with the recent CreateQueue API's being released we are very close to having the ability to provision an entire Contact Center via Terraform (Lambdas, Lex, ConnectFlows the works) I have a number of resources waiting on the merge of #16709

@nbmustafa
Copy link

nbmustafa commented Feb 22, 2021

@gdavison and @maryelizbeth there has been a great contribution from the guys @abebars and @thornleyk to add amazon connect instance and contact follows to the terraform provider, would be awesome if their work can be reviewed. Heaps of eyes on these PRs.
#16709
#16854

@maryelizbeth
Copy link
Contributor

Hi @nashvan - Thanks for raising this! We have been keeping an eye on this request in our backlog but don't have an ETA for review at this time. Once we are able to accommodate these PRs we will reach out to the contributors and set expectations for review.

@abebars
Copy link
Contributor Author

abebars commented May 17, 2021

Hey @maryelizbeth , Thanks for the follow-up on this issue earlier. Any updates on the backlog if we can start at least to review these PRs that @nashvan mentioned. These are just going to be a starter. I love to commit some time to complete all available features for Connect but I want to make sure that we can fast track the process from development to review and merge instead of having a long wait period without seeing progress. thoughts?

@abebars
Copy link
Contributor Author

abebars commented Jun 11, 2021

@maryelizbeth @bflad @ewbankkit I am still not sure, How this will get prioritized, I understand it’s a new resource and require some efforts but all I am looking for is a review and guidance instead of keep blocking the resource from being implemented. I am not sure what else to do or who else to ping to get a single PR reviewed!!

@thornleyk
Copy link
Contributor

@maryelizbeth @bflad @ewbankkit I am still not sure, How this will get prioritized, I understand it’s a new resource and require some efforts but all I am looking for is a review and guidance instead of keep blocking the resource from being implemented. I am not sure what else to do or who else to ping to get a single PR reviewed!!

Agreed, I have a massive amount of tech debt and queued draft resources to push through once this is merged, we are currently automating this outside tf which is a real pain

@abdsahin
Copy link

I am looking forward these PRs to be merged, we are doing a lot of AWS Connect projects integrated with other AWS services and tired of managing them seperately!

@robertstettner
Copy link
Contributor

Could we please get these PRs merged. We would like this available.

@abebars
Copy link
Contributor Author

abebars commented Jul 16, 2021

@maryelizbeth @bflad @ewbankkit Just nudging you again, any thought when this could be a priority for you?

@anGie44
Copy link
Contributor

anGie44 commented Jul 22, 2021

Hi @abebars , just dropping a note here that #16709 has been added to our queue. I don't have an exact ETA of when you'll see a complete review just yet, but the PR's status will change to "In Progress" once it's picked up.

@abebars
Copy link
Contributor Author

abebars commented Jul 28, 2021

Oh Great thanks @anGie44 for the update.

@AdamTylerLynch
Copy link
Collaborator

Update: Added to roadmap:
https://github.com/hashicorp/terraform-provider-aws/blob/main/ROADMAP.md#amazon-connect

@anGie44 anGie44 added the new-data-source Introduces a new data source. label Sep 23, 2021
@abebars
Copy link
Contributor Author

abebars commented Sep 24, 2021

This functionality has been released in v3.60.0 of the Terraform AWS Provider.

@abebars abebars closed this as completed Sep 24, 2021
@github-actions
Copy link

github-actions bot commented Jun 6, 2022

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-data-source Introduces a new data source. new-resource Introduces a new resource. service/connect Issues and PRs that pertain to the connect service.
Projects
None yet
Development

No branches or pull requests