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

Add example tags #236

Merged
merged 1 commit into from
Feb 28, 2020
Merged

Add example tags #236

merged 1 commit into from
Feb 28, 2020

Conversation

lixue323
Copy link
Contributor

No description provided.

- eni
- eip
- image
- slb
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个里面不需要支持这么多的资源,把 eni,image,vswitch都去掉吧

@@ -0,0 +1,15 @@
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

目录名就改为 tags,下面有两个文件add.yml 和 remove.yml

@lixue323 lixue323 changed the title Add example alicoud-tags Add example tags Feb 27, 2020
state: absent
resource_type: 'eni'
resource_ids: '{{ enis.ids }}'
tags: '{{ eni_tags }}'
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个 eni 还在啊

@@ -0,0 +1,121 @@
---
- name: Remove tags for resource
Copy link
Collaborator

Choose a reason for hiding this comment

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

忽然发现这个playbook中没有创建这些要打标的资源啊

@lixue323 lixue323 force-pushed the addtag branch 2 times, most recently from ca33dba to 3efa2a8 Compare February 28, 2020 08:38
vpc_id: '{{ vpcs.vpcs.0.id }}'
register: instances

- name: Creating an ECS instance
Copy link
Collaborator

Choose a reason for hiding this comment

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

创建10台ecs出来打标,这个module中是有count可以支持的,其他的可以只创建一台

name_prefix: '{{ eip_name }}'
register: eips

- name: Changed. Remove eip Tags.
Copy link
Collaborator

Choose a reason for hiding this comment

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

资源的删除单独放到一个文件 sweep.yml 中


group_outboundRules:
- ip_protocol: tcp
port_range: 80/80
Copy link
Collaborator

Choose a reason for hiding this comment

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

安全组规则可以不用要


- Requires Ansible 2.9 or newer
- Requires Ansible-Alicloud 1.16.0 or newer
- Requires footmark 1.17.0 or newer
Copy link
Collaborator

Choose a reason for hiding this comment

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

版本改下


- Requires Ansible 2.9 or newer
- Requires Ansible-Alicloud 1.17.0 or newer
- Requires footmark 1.18.0 or newer
Copy link
Collaborator

Choose a reason for hiding this comment

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

这两个版本不对吧

@xiaozhu36 xiaozhu36 merged commit b1e469a into alibaba:master Feb 28, 2020
@lixue323 lixue323 deleted the addtag branch March 1, 2020 08:45
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.

2 participants