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

aws-ec2 kamelets proposed improvement #516

Closed
valdar opened this issue Sep 16, 2021 · 2 comments
Closed

aws-ec2 kamelets proposed improvement #516

valdar opened this issue Sep 16, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@valdar
Copy link
Member

valdar commented Sep 16, 2021

Since aws-ec2-sink.kamelet.yaml support just one of the possible operations (i.e. createAndRunInstances, startInstances, stopInstances, terminateInstances, describeInstances, describeInstancesStatus, rebootInstances, monitorInstances, unmonitorInstances, createTags, deleteTags) we can create one kamelet for each of those, or just have 2 kamelets:

  • aws-ec2-source.kamelet.yaml: having a structure like from(timer) .... to(aws2-ec2)...to(kamelet:sink) and supporting the following operations:
    • describeInstances
    • describeInstancesStatus
  • aws-ec2-source.kamelet.yaml: having a structure like from(kamelet:source)...to(aws2-ec2) and supporting the following operations:
    • createAndRunInstances
    • startInstances
    • stopInstances
    • terminateInstances
    • rebootInstances
    • monitorInstances
    • unmonitorInstances
    • createTags
    • deleteTags

mind that I have assumed from the operations name what they would do, so someone more familiar with AWS can make more sense of this.

@valdar valdar added the enhancement New feature or request label Sep 16, 2021
@oscerd oscerd self-assigned this Sep 16, 2021
@oscerd
Copy link
Contributor

oscerd commented Sep 16, 2021

Will have a look

@oscerd
Copy link
Contributor

oscerd commented Dec 14, 2021

It would be too complex to have a single kamelet for source and sink, because we'll have to deal with different headers combination. It probably makes more sense to have one kamelet for operation.

@oscerd oscerd closed this as not planned Won't fix, can't repro, duplicate, stale Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants