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

initial orangefs work for sdp #1

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

AJMKing
Copy link

@AJMKing AJMKing commented Jul 20, 2017

Branch for the initial orangefs work for SDP. Contains two new roles to setup and configure Orangefs, with the rpms, and client unit file.

Copy link
Collaborator

@oneswig oneswig left a comment

Choose a reason for hiding this comment

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

Thanks Ali, some good stuff here but we'll need to reshape a bit here and there. I'll try to get the Heat template work done and then I'll try to do a test run of the code checked in.

inventory.pvfs Outdated
@@ -0,0 +1,21 @@
# Ansible Shade uses OpenStack clients running locally
Copy link
Collaborator

Choose a reason for hiding this comment

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

Inventory files should be auto-generated by the Heat role (which you didn't run, so that's fine).

I'm wondering where your DNS is coming from, have you tinkered with that?

What's interesting here is that we have overlapping roles: a node can be both server and client. This needs a little bit of thinking (but probably, in the first instance, manual adjustment)

Copy link
Author

Choose a reason for hiding this comment

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

i had SSH config point to the nodes and sort out the keys. So ansible will be resolving names that way. not sure how this works with heat.

@@ -0,0 +1,10 @@
[Unit]
Copy link
Collaborator

Choose a reason for hiding this comment

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

As discussed, if this could get tucked into the RPM that'd be grand.

@@ -0,0 +1,29 @@
---
- name: copy rpms to server
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think we should have the RPMs as artefacts in this repo, but how we handle them otherwise is TBD. Best option may be to make a locally-accessible repo server which contains the OrangeFS RPMs, plus a few others we've got kicking around.

name: "{{ item }}"
state: present
with_items:
- /tmp/orangefs-2.9.7-1.fc25.x86_64.rpm
Copy link
Collaborator

Choose a reason for hiding this comment

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

These two items appear to be the same...

Copy link
Author

Choose a reason for hiding this comment

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

you meen the two RPMs? one should provide the server only one provides the client.

when: inventory_hostname in groups['orange-data']

- name: setup mdraid 5 on nvmes
command: mdadm --create --verbose /dev/md0 --level=5 --raid-devices=4 /dev/nvme0n1 /dev/nvme1n1 /dev/nvme2n1 /dev/nvme3n1 creates=/dev/md0
Copy link
Collaborator

Choose a reason for hiding this comment

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

If /dev/md0 already exists, does this fail? If it doesn't, it'll get flattened in the next task. If it does, it means the playbook won't rerun.

Copy link
Author

Choose a reason for hiding this comment

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

if md0 exists the command wont run. if it dose not, then it will try to build the array. If the case that md0 was deleted or the mdadm stop, but raid superblocks exist, then the playbook will fail. Should probably brakeout this into a role that sets up mdraid, since lvm is also an option.

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

Choose a reason for hiding this comment

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

It might be tidier syntax to conditionally include task files for client, server, etc. after the common tasks here.

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

Choose a reason for hiding this comment

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

You Emacs users have no shame :-)

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

Choose a reason for hiding this comment

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

Temporary Emacs file?

oneswig pushed a commit that referenced this pull request Mar 27, 2018
First cut at maintenance actions playbook
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