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

Use project manila share user from Azimuth #359

Merged
merged 6 commits into from
Feb 20, 2024
Merged

Conversation

sjpb
Copy link
Collaborator

@sjpb sjpb commented Feb 13, 2024

Changes CaaS project manila shares to use the share user provided by Azimuth (in https://github.com/stackhpc/azimuth/pull/148/files), rather assuming there is only a single access rule. Fixes cases where multiple access rules are defined on the project share.

@sjpb sjpb requested a review from a team as a code owner February 13, 2024 11:23
Copy link
Member

@JohnGarbutt JohnGarbutt left a comment

Choose a reason for hiding this comment

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

@sjpb I am hoping this one gets through the testing. This looks OK to me now. And its now consistent with:
https://github.com/stackhpc/caas-workstation/pull/17/files

@sjpb
Copy link
Collaborator Author

sjpb commented Feb 14, 2024

With azimuth-config on f06894dc56f3f09d76cb7ac3e59f0e3c5c240042 and ci environment configured as shown below this failed to deploy in rcp-cloud-portal-demo with

Failed to create platform. To retry please click patch. Possible reason for the failure was: non-zero return code

config:

# Set the volume-type name for fast volumes
arcus_fast_volume_type: arcus-ceph01-nvme-rbd

# Enable the project Manila share in Azimuth
azimuth_openstack_manila_project_share_gb: 40

# Enable the fast-volume-type variants of the SSH and standard workstation platforms
azimuth_caas_stackhpc_workstation_metadata_url: >-
  https://raw.githubusercontent.com/stackhpc/caas-workstation/{{ azimuth_caas_stackhpc_workstation_git_version }}/ui-meta/workstation-fast-volume-type.yml
azimuth_caas_stackhpc_workstation_ssh_metadata_url: >-
  https://raw.githubusercontent.com/stackhpc/caas-workstation/{{ azimuth_caas_stackhpc_workstation_git_version }}/ui-meta/workstation-external-ssh-fast-volume-type.yml
azimuth_caas_stackhpc_workstation_extra_vars_overrides:
  cluster_volume_type_fast: "{{ arcus_fast_volume_type }}"

# Slurm appliance: home on fast volume
azimuth_caas_stackhpc_slurm_appliance_git_version: feat/caas-manila-share-user
azimuth_caas_stackhpc_slurm_appliance_metadata_url: >-
  https://raw.githubusercontent.com/stackhpc/ansible-slurm-appliance/{{ azimuth_caas_stackhpc_slurm_appliance_git_version }}/environments/.caas/ui-meta/slurm-infra-fast-volume-type.yml
azimuth_caas_stackhpc_slurm_appliance_extra_vars_overrides:
  home_volume_type_fast: "{{ arcus_fast_volume_type }}"
  cluster_home_manila_share: false

# Slurm appliance: home on manila
azimuth_caas_cluster_templates_overrides:
  slurm-project-manila:
    gitUrl: "{{ azimuth_caas_stackhpc_slurm_appliance_git_url }}"
    gitVersion: "{{ azimuth_caas_stackhpc_slurm_appliance_git_version }}"
    uiMetaUrl: >-
      https://raw.githubusercontent.com/stackhpc/ansible-slurm-appliance/{{ azimuth_caas_stackhpc_slurm_appliance_git_version }}/environments/.caas/ui-meta/slurm-infra-manila-home.yml
    playbook: "{{ azimuth_caas_stackhpc_slurm_appliance_playbook }}"
    jobTimeout: "{{ azimuth_caas_stackhpc_slurm_appliance_job_timeout_seconds }}"
    extraVars:
      cluster_image: "{{ community_images_image_ids.openhpc }}"
      cluster_home_manila_share: true # is default if cluster_project_manila_share anyway
      cluster_home_manila_share_type: ceph01_cephfs
    jobTimeout: "{{ azimuth_caas_stackhpc_slurm_appliance_job_timeout_seconds }}"
    envVars:
      ANSIBLE_INVENTORY: environments/common/inventory,environments/.caas/inventory

@sjpb sjpb merged commit be06d3d into main Feb 20, 2024
1 check passed
@sjpb sjpb deleted the feat/caas-manila-share-user branch February 20, 2024 09:59
@JohnGarbutt JohnGarbutt added the bug Something isn't working label Feb 20, 2024
@sjpb sjpb restored the feat/caas-manila-share-user branch February 22, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants