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

OSASINFRA-3495: Adding a StrageNFS Network to work with Manila/Ganesha and Ceph #228

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

MiguelCarpio
Copy link

Adding a StrageNFS Network to work with Manila/Ganesha and Ceph

@MiguelCarpio
Copy link
Author

/hold

Copy link
Author

Choose a reason for hiding this comment

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

Here, I'm creating the StorageNFS network using the default datacentre provider and the VLAN 70

Copy link
Author

Choose a reason for hiding this comment

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

Here, I'm defining the StorageNFS network to be used by Ceph

Copy link
Author

Choose a reason for hiding this comment

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

Updating the standalone_parameters.ceph_ansible.yaml to use the hostonly_nfs_ganesha defined defaults.yaml

@MiguelCarpio MiguelCarpio deleted the OSASINFRA-3495 branch June 26, 2024 17:02
@MiguelCarpio MiguelCarpio restored the OSASINFRA-3495 branch June 26, 2024 17:05
@MiguelCarpio MiguelCarpio reopened this Jun 26, 2024
@MiguelCarpio MiguelCarpio changed the title OCPBUGS-33424: Adding a StrageNFS Network to work with Manila/Ganesha and Ceph OSASINFRA-3495: Adding a StrageNFS Network to work with Manila/Ganesha and Ceph Jun 26, 2024
@EmilienM
Copy link
Contributor

from a very quick look, this sounds fine.
have you tested it?

@MiguelCarpio
Copy link
Author

MiguelCarpio commented Jun 27, 2024

from a very quick look, this sounds fine. have you tested it?

@EmilienM. Yes, I've tested it but I always get this error at TASK | Render hieradata from template:

2024-06-26 18:06:53.030809 | 78ac448c-2b24-54fe-e7f7-000000000490 | FATAL | Render hieradata from template | dell-r640-100 | item=vip_data | error={"ansible_loop_var": "item", "changed": false, "item": "vip_data", "msg": "AnsibleUndefinedVariable: 'dict object' has no attribute 'storage_nfs'. 'dict object' has no attribute 'storage_nfs'"}

@@ -46,6 +46,18 @@ resource_registry:
{% endif %}
parameter_defaults:
CloudName: {{ hostname }}.{{ clouddomain }}
# Enable ganesha
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be conditional whether manila is enabled

@EmilienM
Copy link
Contributor

I think during the tripleo deploy (after ceph itself), because you need to load a network file that has the StorageNFS definition. You have it for Ceph, but not for the tripleo deployment after.
Grep for /usr/share/openstack-tripleo-heat-templates/network_data_undercloud.yaml, we load that file when doing tripleo deploy. We could template this file and optionally add the StorageNFS if manila is enabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants