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

Feature Request: AppStream support #6508

Closed
8 tasks done
glucas opened this issue Nov 17, 2018 · 35 comments
Closed
8 tasks done

Feature Request: AppStream support #6508

glucas opened this issue Nov 17, 2018 · 35 comments
Assignees
Labels
new-resource Introduces a new resource. service/appstream Issues and PRs that pertain to the appstream service.

Comments

@glucas
Copy link

glucas commented Nov 17, 2018

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 "me too" comments, 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

Add support for AppStream resources like fleets, stacks, and image builders.

These resources are now supported by CloudFormation

There are still gaps in AppStream automation -- images must be configured manually -- but being able to create the supported resources via terraform would be a big help.

New Resources

@bflad bflad added new-resource Introduces a new resource. service/appstream Issues and PRs that pertain to the appstream service. labels Nov 19, 2018
@sahil-lakhwani
Copy link

I am interested to work on this.

@devonbleak
Copy link
Contributor

Seems like we should also include aws_appstream_directory_config

@UVADave
Copy link

UVADave commented Jun 10, 2019

I am using https://github.com/ops-guru/terraform-provider-appstream but it is a little rough around the edges and may not be compatible with terraform 0.12. While we (University of Virginia Library) are only using ApStream for a single application, we have been quite happy with it and expect more adoption... an up to date provider would be great.

@wrschneider
Copy link
Contributor

wrschneider commented Jun 10, 2019

Another plus would be a hypothetical aws_appstream_stack should return the automatically created S3 bucket for app settings so you can attach a managed bucket policy via Terraform as well.

@fred97855
Copy link

Has there been any movement on this? Would be great to see it included in the AWS provider

@aablake1
Copy link

Has this been added to the aws provider, if so is there any documentation on how to use it?

@ignatev
Copy link

ignatev commented May 12, 2020

@ewbankkit Could you please help us with instructions on how to use terraform with appstream?

@Japeth
Copy link

Japeth commented Jul 7, 2020

I can't find any documentation on using appstream with terraform. Has this actually been added?

@edg4rgarci4
Copy link

I can't find any documentation on using appstream with terraform. Has this actually been added?

Any updates?

@wrschneider
Copy link
Contributor

I am still interested in this... we have a TF module that creates a stack + fleet + autoscaling groups but has to rely on CloudFormation for the stack/fleet. :-( it would be a win to have even minimal support in TF for the three resources (stack, fleet, association).

Has anyone made an attempt at a PR yet?

@wrschneider
Copy link
Contributor

I started to get the ball rolling #14675, using the ops-guru provider as a starting point.
I'm not fluent in Go and could use help.

@wrschneider
Copy link
Contributor

OK, I now have basic support for a stack and fleet, copied heavily from ops-guru and cleaned up a bit.
It's still rough around the edges but IMO even in its current state it would be better than what I'm doing today.

@aablake1
Copy link

aablake1 commented Aug 17, 2020

@wrschneider Were you able to update ops-guru's provider to include adding an iam role to an image builder? I tried taking a stab at it but Go isn't a language I'm familiar with. Right now I'm just running some powershell from within terraform to build the image builder instance. The stack and fleet seem ok.

@wrschneider
Copy link
Contributor

@aablake1 I wasn't even trying to solve for the image builder. The stack and fleet are the parts I really need to automate. In my typical workflow I'm setting up the image builder and creating images by hand in non-production environments, then sharing them.

What I have in this PR now is the bare minimum I need to replace Cloudformation in my TF for stack/fleet creation.

@aablake1
Copy link

@wrschneider makes sense. It would be nice if they can add support for AppStream to the main aws terraform provider. Ops-guru provider is better than nothing though so making do with that for now.

@wrschneider
Copy link
Contributor

@aablake1 that was the whole point of this issue/my PR, to incorporate into the main AWS TF provider. If I have to use another TF provider I'm not really that much better off than using CF inside TF.

Now I have all the lint checks passing, at least.

@arnvid
Copy link

arnvid commented Oct 15, 2020

@wrschneider - I am not sure which one of the AppStream providers you used as your source - but we also based ours on ops-guru when we collected the different once into this one https://github.com/arnvid/terraform-provider-appstream currently at v1.0.8 and we are working on a v1.0.9 to work around some problems we found in the provider when updating stack. If you are using the original ops-guru one we have some problems using it in production environments.

One would hope that Appstream finds it way into the AWS provider - but if does not happen soon I have been thinking of publishing the one I help work on to registry.

@wrschneider
Copy link
Contributor

@arnvid yes I based mine on ops-guru as well. It seemed like a reasonable start and there were only some minor tweaks I had to make to get it to work.

@ismailyenigul
Copy link

Hi @arnvid @wrschneider
Which repo can we use until 15609 and 15610 PR merged? Can we use https://github.com/arnvid/terraform-provider-appstream in prod?
@arnvid Do you have a plan a fix for the following issue in near time? And what is the exact issue you are experiencing when updating stack?
we are working on a v1.0.9 to work around some problems we found in the provider when updating stack

@flickerfly
Copy link

I see it has been a few months. Have the plans to merge this into the aws provider become any more solid?

@wrschneider
Copy link
Contributor

Haven't touched it since then. Is this even on Hashicorp's backlog?

@fishdood
Copy link

fishdood commented Jun 1, 2021

I came here to make the same comment. This would be really valuable to implement.

@kodyabbott
Copy link

@breathingdust could we get AppStream added?

@rednevals
Copy link

I'm in need of this as well...

@breathingdust
Copy link
Member

Hi all! 👋 AppStream is on the roadmap for this quarter:

@wrschneider
Copy link
Contributor

Hi all! 👋 AppStream is on the roadmap for this quarter:

Yay! @breathingdust Let me know how I can help

@YakDriver YakDriver self-assigned this Aug 24, 2021
@YakDriver YakDriver added this to the v3.56.0 milestone Aug 24, 2021
@breathingdust breathingdust removed this from the v3.56.0 milestone Aug 26, 2021
@liberops
Copy link

Is there a way to associate an Appstream stack with an Appstream fleet with Terraform?
Thanks

@rednevals
Copy link

rednevals commented Sep 30, 2021

Is there a way to associate an Appstream stack with an Appstream fleet with Terraform? Thanks

@liberops Yup!
arnvid/appstream

terraform {
  required_providers {
    appstream = {
      source  = "arnvid/appstream"
      version = "2.0.0"
    }
  }
)

resource "appstream_stack_attachment" "stack_fleet_association" {
  appstream_stack_id = appstream_stack.my_stack.name
  appstream_fleet_id = appstream_fleet.my_fleet.name
}

@etiennejeanneau
Copy link

First off, thanks for the work on this!
It seems AssociateFleet https://docs.aws.amazon.com/sdk-for-go/api/service/appstream/#AppStream.AssociateFleet is missing on the roadmap to glue the stack and fleet together. I'm unsure if it has been missed in the official hashicorp roadmap.
*DirectoryConfig would be nice too ;)

@chrobotm
Copy link

chrobotm commented Oct 7, 2021

@breathingdust ☝️

@breathingdust
Copy link
Member

breathingdust commented Oct 20, 2021

Hi @mbotmcc, it's coming! Edited the issue description to include the resources we are currently planning to add.

@chrobotm
Copy link

#21718 is a bit of an issue for us as we use desktop mode

@Arisfx
Copy link

Arisfx commented Jan 10, 2022

Hi team, great work on this! i cant find the terraform code for image registry- am i missing something?

@breathingdust
Copy link
Member

Closing this issue as the originally scoped work has been completed. Created a separate issue to track aws_appstream_image in #22536

@github-actions
Copy link

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 May 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-resource Introduces a new resource. service/appstream Issues and PRs that pertain to the appstream service.
Projects
None yet
Development

No branches or pull requests