coverage: 77.835%. remained the same
when pulling 97d56c7 on zhekazuev:feature/add-db-instance-access-field
into 8587253 on gophercloud:master.

@EmilienM Hi,

What better to use for Access field, a pointer(*AccessOpts) or a value(AccessOpts)?
Commit with changes: 0b8b1e1

From the risks and style side.
I prefer to not use pointers where possible, to avoid null pointer issues, but maybe for current scenario there is no option with a value.

@EmilienM Hi,

What better to use for Access field, a pointer(*AccessOpts) or a value(AccessOpts)? Commit with changes: 0b8b1e1

From the risks and style side. I prefer to not use pointers where possible, to avoid null pointer issues, but maybe for current scenario there is no option with a value.

@EmilienM Hi,

Could you review these changes?

I just added new field - Access. Without methods specific for that field, e.g.:

• UpdateInstanceAccessbility - update access options like here.

And decided to use the value for Access struct instead of pointers as described below.
If it is important to use pointers, I can restore the changes to the pointer version.