sureshanaparti commented on code in PR #10271:
URL: https://github.com/apache/cloudstack/pull/10271#discussion_r3071308682
##########
plugins/storage/volume/flasharray/src/main/java/org/apache/cloudstack/storage/datastore/adapter/flasharray/FlashArrayAdapterFactory.java:
##########
@@ -25,7 +25,7 @@ public class FlashArrayAdapterFactory implements
ProviderAdapterFactory {
@Override
public String getProviderName() {
- return "Flash Array";
+ return "FlashArray";
Review Comment:
@sdodsley this is persisted for storage pools, so existing pool with
storage_provider_name: 'Flash Array' needs to be updated with 'FlashArray'.
(Add sql: `UPDATE cloud.storage_pool SET storage_provider_name = 'FlashArray'
WHERE storage_provider_name = 'Flash Array';` in upgrade file :
_engine/schema/src/main/resources/META-INF/db/schema-42200to42210.sql_)
```
mysql> SELECT id, name, pool_type, storage_provider_name FROM
cloud.storage_pool;
+----+----------------------------------------------------------+-------------------+-----------------------+
| id | name | pool_type
| storage_provider_name |
+----+----------------------------------------------------------+-------------------+-----------------------+
| 1 | ref-trl-11538-k-Mol8-suresh-anaparti-kvm-pri1 |
NetworkFilesystem | DefaultPrimary |
| 2 | ref-trl-11538-k-Mol8-suresh-anaparti-kvm-pri2 |
NetworkFilesystem | DefaultPrimary |
| 3 | ref-trl-11538-k-Mol8-suresh-anaparti-kvm2-local-983fc8c4 | Filesystem
| DefaultPrimary |
| 4 | ref-trl-11538-k-Mol8-suresh-anaparti-kvm1-local-e38cd5cf | Filesystem
| DefaultPrimary |
| 5 | pflexcspool1 | PowerFlex
| PowerFlex |
| 6 | pflexcspool2 | PowerFlex
| PowerFlex |
| 7 | pflexsbpool | PowerFlex
| PowerFlex |
+----+----------------------------------------------------------+-------------------+-----------------------+
7 rows in set (0.00 sec)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
