Skip to content

Documentation bug for security groups #345

Open
Open
Documentation bug for security groups#345

Description

@palaueb
palaueb
opened on Feb 4, 2022

On the link https://php-opencloudopenstack.readthedocs.io/en/latest/services/compute/v2/servers.html#security-groups

the documentation says that we set the security groups by this way:

$options['securityGroups'] = ['secGroup1', 'default', 'secGroup2'];

But it returns error, it may be this way:

$options['securityGroups'] = [
   ['name'=>'secGroup1'],
   ['name'=>'default'],
   ['name'=>'secGroup2']
];

Can we change it? thanks!!

Activity

  1. k0ka commented on Dec 12, 2023

    @k0ka
    k0ka
    on Dec 12, 2023
    Member

    Hello.

    1. our latest documentation is on https://php-openstack-sdk.readthedocs.io/en/latest/services/compute/v2/servers.html#security-groups
    2. It looks like you're right (according to https://docs.openstack.org/api-ref/compute/), but we have to create corresponding test. We also have security_groups defined as array of uuids in https://github.com/php-opencloud/openstack/blob/master/src/Networking/v2/Params.php#L275 which is most probably wrong
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions