Skip to content

Conversation

@jwerak
Copy link
Member

@jwerak jwerak commented Oct 26, 2020

No description provided.

@jwerak jwerak requested review from jskorzyn and rwngwn October 26, 2020 08:17
README.md Outdated
Additional Container Group on OpenShift
- `oc create -f ocp-setup/role-pod-manager.yml`
- `oc create -f ocp-setup/sa-tower-container-group.yml`
- `oc create -f ocp-setup/rb-tower-container-group.yml`
Copy link
Member

Choose a reason for hiding this comment

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

why not oneliner?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is just WIP documentation, I though this is more readable.
In future I would like to code it all so that we can set it up from scratch on new OCP Cluster.

Copy link
Member

Choose a reason for hiding this comment

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

even as oneliner you can use -f multiple times for one oc command IIRC

gcp_compute_instance:
state: present
name: "{{ vm_name | default('ansible-tower-test', true) }}"
machine_type: n1-standard-1
Copy link
Member

Choose a reason for hiding this comment

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

hardoced machine type

Copy link
Collaborator

Choose a reason for hiding this comment

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

+1
Should be set as variable

Copy link
Collaborator

Choose a reason for hiding this comment

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

I will add empty line between tasks. Just to make code more clean and

Copy link
Member Author

Choose a reason for hiding this comment

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

This is for the PoC part to demonstrate that we can run VM in GCP. I don't know if we want to parametrize machine type. I would keep it as default for no wand parametrize it once we define a set of values we want to be configurable for all clouds.
WDYT?

Copy link
Member

Choose a reason for hiding this comment

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

if you keep it hardoced then create an issue please. otherwise we can easily carry it.

Jakub Veverka added 2 commits October 30, 2020 10:14
mainly documentaiton of how to set OpenShift and Tower.
Custom container image had to be used because there gcp modules don't run on
default Ansible Tower instance group.
@jwerak
Copy link
Member Author

jwerak commented Oct 30, 2020

I've removed the Ansible and OCP related content.
The content is migrated to main ceenter repo as part of this PR: ceenter/ceenter#5

@jwerak jwerak requested a review from rwngwn November 5, 2020 15:14
@jwerak jwerak changed the title configure ansible tower to use execution container group Migrate Ansible playbooks and roles to ceenter/ceenter Nov 5, 2020
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