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

need help adding ability to have tag support for ESdomain #716

Closed
wants to merge 3 commits into from

Conversation

IndushaS
Copy link

@IndushaS IndushaS commented Dec 3, 2021

Hello I want to add tag support for elasticsearchservice domain but I am getting an error for line 33 in elasticsearchservice-domain.go domain.Tags undefined (type *elasticsearchservice.DomainInfo has no field or method Tags). Any help would be appreciated as I am new to this repo.

@IndushaS IndushaS requested a review from a team as a code owner December 3, 2021 01:40
@der-eismann
Copy link
Member

Unfortunately not every resource is returning their tags immediately. For this service you need to request the tags separately via the ListTags function.

@ekristen
Copy link
Collaborator

This has been implemented in the actively managed fork of aws-nuke

If you have a chance, please check it out and let us know if you run into an issues by opening an issue over on the fork.


Please see the copy of the notice from the README about the deprecation of this project. Sven was kind enough to grant me access to help triage and close issues and pull requests that have already been addressed in the actively maintained fork. Some additional information is located in the welcome issue for more information.

Caution

This repository for aws-nuke is no longer being actively maintained. We recommend users to switch to the actively maintained fork of this project at ekristen/aws-nuke.
We appreciate all the support and contributions we've received throughout the life of this project. We believe that the fork will continue to provide the functionality and support that you have come to expect from aws-nuke.
Please note that this deprecation means we will not be addressing issues, accepting pull requests, or making future releases from this repository.
Thank you for your understanding and support.

@ekristen ekristen closed this Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants