feat(Cloud Databases): Database user password complexity validation #4931

alexhemard · 2023-11-16T20:10:08Z

From the IBM Cloud Announcement:

IBM Cloud Databases: User Password Complexity Policy Change
Please note, effective November 15, 2023, IBM Cloud Databases is changing our password complexity policy for new or changed database user passwords: Any database user passwords edited or created will be required to be a minimum of 15 characters and contain at least one letter and number.

This PR validates the updated database user password complexity during plan

Update documentation
Change minimum password length to 15
Validator for adminpassword attribute
Validator in CustomizeDiff method for users attribute
Handle ops_manager user updates by deleting then re-creating

Other changes:

Adds DatabaseUser struct with Password validation and CRUD methods
Refactor userChange struct to use *DatabaseUser
Replaced userUpdateCreate method with separate calls to Update() and Create()
Renamed check<Thing> methods to validate<Thing>
Renamed customized diff methods to validate<Thing>Diff

Community Note

Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #4951

Output from acceptance testing:

TESTARGS='-run="TestValidateUserPassword"' make test

go test -run=TestValidateUserPassword -timeout=30s -parallel=4 github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/database github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/directlink github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/dnsservices github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/enterprise 
ok  	github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/database	0.260s

$ make testacc TESTARGS='-run="(TestAccIBMDatabaseInstancePostgresBasic|TestAccIBMMongoDBEnterpriseDatabaseInstanceBasic)"'

...

=== RUN   TestAccIBMMongoDBEnterpriseDatabaseInstanceBasic
=== PAUSE TestAccIBMMongoDBEnterpriseDatabaseInstanceBasic
=== RUN   TestAccIBMDatabaseInstancePostgresBasic
=== PAUSE TestAccIBMDatabaseInstancePostgresBasic
=== CONT  TestAccIBMMongoDBEnterpriseDatabaseInstanceBasic
=== CONT  TestAccIBMDatabaseInstancePostgresBasic
--- PASS: TestAccIBMDatabaseInstancePostgresBasic (1150.60s)
--- PASS: TestAccIBMMongoDBEnterpriseDatabaseInstanceBasic (6419.03s)
PASS

…lexity

alexhemard · 2023-11-22T18:18:10Z

ibm/service/database/resource_ibm_database.go

-	return nil
-}
-
-func checkGroupValue(name string, limits GroupResource, divider int, diff *schema.ResourceDiff) error {


This appears unused

omaraibrahim · 2023-11-27T15:19:52Z

ibm/service/database/resource_ibm_database.go

-	delay := time.Tick(delayDuration)
+	ticker := time.NewTicker(delayDuration)
+	delay := ticker.C
+	defer ticker.Stop()


Why are we changing the delay logic?

This got left in while debugging waitForDatabaseTaskComplete but allows the Ticker to be garbage collected.
See https://pkg.go.dev/time#Tick

omaraibrahim · 2023-11-27T15:22:04Z

ibm/service/database/resource_ibm_database.go

+			return fmt.Errorf("%s group %s must be >= %d and <= %d in increments of %d", groupId, resourceName, resource.Minimum/nodeCount, resource.Maximum/nodeCount, resource.StepSize/nodeCount)
+		}
+	}
+	if value != resource.Allocation/nodeCount && !resource.IsAdjustable {


We may want to add comments on top of each of these if statements. It's a little difficult to see why these are needed without having a strong understanding of the way we manage scaling.

omaraibrahim

LGTM

obai-1 · 2023-11-27T21:33:03Z

ibm/service/database/resource_ibm_database.go

+func (c *userChange) isDelete() bool {
+	return c.Old != nil && c.New == nil
+}
+
+func (c *userChange) isCreate() bool {
+	return c.Old == nil && c.New != nil
+}
+
+func (c *userChange) isUpdate() bool {
+	return c.New != nil && c.Old != nil && c.Old.Password != c.New.Password
+}


This considers password change as an update. What about changes to name/type/role ?

I understand some of them will cause delete and recreate. How are they handled?

…b-password-complexity

ocofaigh · 2023-12-05T10:35:55Z

I don't see this in the 1.60.0 release notes, so I'm guessing it will be part of 1.61.0 ?

- Add VM no storage to deployment type Fixed catalog service extensions and values metadata params of workspace ds (#4957) update CHANGELOG Bump up version to 1.60.0 feat(Cloud Databases): Database user password complexity validation (#4931) * feat(Cloud Databases): Database user password complexity validation * update tests to conform to new complexity rules * fix password validation edgecases * add test case for password beginning with special char * fix regexp * refactor of db users * delete and create ops manager users when updating * validate adminpassword, update docs Add new service Logs Router to IBM Terraform Provider (#4945) * Added new service Logs Router to IBM Terraform Provider * Ran detect-secrets and re-generated secrets.baseline * Added emptycredentialsError check for Logs Router Client Error Signed-off-by: Ian Reyes <[email protected]> * Removed redundant general provider set up information. Fixed Service Subcategories Signed-off-by: Ian Reyes <[email protected]> --------- Signed-off-by: Ian Reyes <[email protected]> add virtual optical device update err msg Add missing changes Update go.mod (#4963) - Update go.mod: bluemix-go version Fix IBM i documentation bug (#4969) * Fix IBM i documentation bug On several documentation pages, it incorrectly implied that IBMi was the only usable OS for that resource/data source. * Update help URLs Regenerate Projects TF to fix generated doc and samples (#4961) * fix integration test * fix integration test * fix integration test * fix name * pr comments * pr comments * fix doc * fix doc * new version * new version * new version * new version * update code to the latest SDK level * fix integration tests * fix integration tests * fix integration tests * fix integration tests * address PR comment * typo * typo * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * add wait for create and delete * add wait for create and delete * . * fix integration tests * fix integration tests * fix integration tests * fix integration tests * fix integration tests * regenerate * new code generation * new generated code * point to latest sdk * point to latest sdk * adjust test cases and doc * adjust samples * update to latest node sdk * merge with master * Update project_instance.html.markdown Fixing formatting issues * Update project_event_notification.html.markdown * Update README.md * Update variables.tf * Update project_event_notification.html.markdown * new version * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * feat: projects graph fragment pattern * align with origin/master * update test cases * restore README.md and versions.tf to master branch * remove duplicated entries * feat: graph fragment pattern poc * feat: restore definition blocks * regenerate the terraform code based on latest YAML * regenerate the terraform code based on latest YAML * regenerate the terraform code based on latest YAML * update doc * update doc * fix test cases * commit generated code * commit fix * commit fix * test cases * test cases * test cases * test cases * updates * upgrade to latest go sdk * update generated doc * update generated code * updated secrets baseline * update code to latest API changes * secrets * fix secrets * update samples with real values * update secrets * update secrets * update generated code to the latest API definition * update secrets baseline * merge with master * merge with master * merge with master * update code to the latest SDK level * address some review comments * address PR comments * address PR comments * update to latest sdk level * update to latest sdk level * update to latest sdk level * merge with master * update to the latest sdk level * format test case * regenerate the code with generator fixes * regenerate code * fix "inputs" and "settings" properties --------- Co-authored-by: dvesperini <[email protected]> Co-authored-by: knee-na <[email protected]> Co-authored-by: Mark-Kulube <[email protected]> Removed Logs Router Provider Signed-off-by: Ian Reyes <[email protected]> Update pi_user_data to accept string input (#4974) * Add string support for pi_user_data field in instance creation * Add new user data test * Updated pi_user_data documentation * Remove duplicate import dependabot: bump actions/setup-go from 4 to 5 Bumps [actions/setup-go](https://github.com/actions/setup-go) from 4 to 5. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@v4...v5) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> new host link endpoint commits (#4970) * new host link endpoint commits * add host link agent argument and example. Fixed RHEL7 repos to RHEL8 --------- Co-authored-by: Justin Kanz <[email protected]> Add mtu and accessConfig flags to subnet create commands for terraform (#4690) * Add mtu and accessConfig flags to subnet create commands for terraform * Fix terraform compile errors * Fix terraform compile error from yaml changes * Clean up warnings in powervs provider * Fix bug checking region instead of zone for satellite check * Hide shared processor pool and IBMi license arguments from being set in satellite location * Fix typing issues in MTU and access config Three issues here: - There is no schema.TypeInt64, so I have to set this field to schema.TypeInt. Pretend this value is an int and then cast it back to an int64. This an issue with the terraform sdk. - I accidentally set schema.Int on access config where it should be schema.TypeString - All passed in variables by the user need to be set or else the sdk and/or service broker can't safeguard against bad values correctly * Update new network field satellite documentation * Revert nullable name change Due to incompatibility between to the two service brokers, this change has to be undone to maintain consistency. * Update go mod files for new sdk version * Add satellite network acceptance test * Update mtu and jumbo descriptions * Add access-config validation to terraform feat: add new service mqcloud Service Credentials secret addition (#4926) * SC addition * SC addition * SC addition * update function updated * SC unit tests added * SC unit tests added * d * tests fixes * tests fixes * update sdk * .secrets.baseline update * .secrets.baseline update * .secrets.baseline update * Update sm_service_credentials_secret_metadata.html.markdown --------- Co-authored-by: Yonathan-Yellin <[email protected]> Co-authored-by: Avi Ribchinsky <[email protected]> Co-authored-by: Tatyana <[email protected]> Co-authored-by: Idan Adar <[email protected]> Feature is snapshot consistency group and Migration PR (#4973) * GO-SDK Updated * ibm_is_snapshot_consistency_group development * Consistency Group changes to Snapshot * Documentation changes for snapshot * SDK changes * Bankup Policy Jobs Changes * included_content development * match_resource_type development * Implemented Documentation Fix * PR review comments incorporated * SDK update * Updated to released SDK Fix typo in filename fix detect secret marking false positive feat(Cloud Databases): Redis Database User RBAC support (#4982) Update version to 1.5.6 fix(Cloud Databases): fix Unwrap return value for go 1.18 compat (#4991) Signed-off-by: Alex Hemard <[email protected]> Address comments update power version update issue fixed ibm_is_subnet_reserved_ip (#4988) Adding Flexible IOPS (#4992) * - Add IOPs to volume - Update documentation - Make strorageType required for COS import - Update Volume Create Disk Type Required - Update Create Instance Storeage Type to Required - vol_upd_iops_level * Update version to 1.5.6 * update documentation * update power version --------- Co-authored-by: michael kad <michaelkad> Removing Support For Power VPN Create (#4993) * Add Deprectated for VPN Create * Update text * Update version to 1.5.6 * Update version to 1.5.6 * update power version --------- Co-authored-by: michael kad <michaelkad> Feature(share-crr): Share cross region replication (#4995) * feature(share-crr): Share cross region replication * sdk updates * updated last sync options * updated sdk and latest_sync * updated schema and doc * tf example * doc and schema correction * acceptance test * sdk changes * updated sdk * added nil check * doc link added * remove common folder sdk * doc update Add variable to ok address review comments dependabot: bump golang.org/x/crypto from 0.16.0 to 0.17.0 Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.16.0 to 0.17.0. - [Commits](golang/crypto@v0.16.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Enhancement: Added operating system attributes to is images data sources (#4998) * Enhancement: Added operating system attributes to is images data sources * set the values * test and doc changes added enhancement to one step delegate resolver in is_vpc resolved delete issue for the floated nics on bm server Bump up versio to 1.61.0-beta0 Regenerate projects provider based off the latest go sdk (#5003) * new version * new version * new version * update code to the latest SDK level * fix integration tests * fix integration tests * fix integration tests * fix integration tests * address PR comment * typo * typo * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * add wait for create and delete * add wait for create and delete * . * fix integration tests * fix integration tests * fix integration tests * fix integration tests * fix integration tests * regenerate * new code generation * new generated code * point to latest sdk * point to latest sdk * adjust test cases and doc * adjust samples * update to latest node sdk * merge with master * Update project_instance.html.markdown Fixing formatting issues * Update project_event_notification.html.markdown * Update README.md * Update variables.tf * Update project_event_notification.html.markdown * new version * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * feat: projects graph fragment pattern * align with origin/master * update test cases * restore README.md and versions.tf to master branch * remove duplicated entries * feat: graph fragment pattern poc * feat: restore definition blocks * regenerate the terraform code based on latest YAML * regenerate the terraform code based on latest YAML * regenerate the terraform code based on latest YAML * update doc * update doc * fix test cases * commit generated code * commit fix * commit fix * test cases * test cases * test cases * test cases * updates * upgrade to latest go sdk * update generated doc * update generated code * updated secrets baseline * update code to latest API changes * secrets * fix secrets * update samples with real values * update secrets * update secrets * update generated code to the latest API definition * update secrets baseline * merge with master * merge with master * merge with master * update code to the latest SDK level * address some review comments * address PR comments * address PR comments * update to latest sdk level * update to latest sdk level * update to latest sdk level * merge with master * update to the latest sdk level * format test case * regenerate the code with generator fixes * regenerate code * fix "inputs" and "settings" properties * regenerated doc and samples * fix samples * terraform * terraform * terraform * terraform * terraform --------- Co-authored-by: dvesperini <[email protected]> Co-authored-by: knee-na <[email protected]> Co-authored-by: Mark-Kulube <[email protected]> Feature(vpc Advertise routes): Support route advertising in vpc (#5005) * added advertised route support * doc updates for routing table/s * Change advertise_routes_to from List to HashSet and modify create, update and read methods accordingly * Add latest generated go sdk * update with latest sdk * add beta maturity to sdk * corrected docs as per review comment * updated go mod * Update resource_ibm_is_bare_metal_server.go * rm common --------- Co-authored-by: Bhavesh Shrivastav <[email protected]> Co-authored-by: Ujjwal Kumar <[email protected]> Bugs fixes for Secrets Manager (#5008) * SC addition * SC addition * SC addition * update function updated * SC unit tests added * SC unit tests added * d * tests fixes * tests fixes * update sdk * .secrets.baseline update * .secrets.baseline update * .secrets.baseline update * Update sm_service_credentials_secret_metadata.html.markdown * bugs fixes * bugs fixes * bugs fixes --------- Co-authored-by: Yonathan-Yellin <[email protected]> Co-authored-by: Avi Ribchinsky <[email protected]> Co-authored-by: Tatyana <[email protected]> Co-authored-by: Idan Adar <[email protected]> dependabot: bump crazy-max/ghaction-import-gpg from 6.0.0 to 6.1.0 Bumps [crazy-max/ghaction-import-gpg](https://github.com/crazy-max/ghaction-import-gpg) from 6.0.0 to 6.1.0. - [Release notes](https://github.com/crazy-max/ghaction-import-gpg/releases) - [Commits](crazy-max/ghaction-import-gpg@v6.0.0...v6.1.0) --- updated-dependencies: - dependency-name: crazy-max/ghaction-import-gpg dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> fix: add multipart support for mqcloud service (#5010) * fix: multipart fix for mqcloud (cherry picked from commit e7253421db30943d3582ce07d9504a2117f30ce1) * enable enforceReservedDeploymentPlan --------- Co-authored-by: Shuaib Bapputty <[email protected]> added a nil check for boottarget of bms Delete wait logic changes Changes wrt latest schematics sdk Revert VPN Changes, will go in at end Q1 SM docs bug fix (#5018) * SC addition * SC addition * SC addition * update function updated * SC unit tests added * SC unit tests added * d * tests fixes * tests fixes * update sdk * .secrets.baseline update * .secrets.baseline update * .secrets.baseline update * Update sm_service_credentials_secret_metadata.html.markdown * bugs fixes * bugs fixes * bugs fixes * docs bugs fixes --------- Co-authored-by: Yonathan-Yellin <[email protected]> Co-authored-by: Avi Ribchinsky <[email protected]> Co-authored-by: Tatyana <[email protected]> Co-authored-by: Idan Adar <[email protected]> fix(docs): UI-33634: mention correct image name in image data source example Bump up version to 1.61.0 update go version to 1.19 CD scc doc updates Latest SCC description updates dependabot: bump github.com/cloudflare/circl from 1.3.3 to 1.3.7 Bumps [github.com/cloudflare/circl](https://github.com/cloudflare/circl) from 1.3.3 to 1.3.7. - [Release notes](https://github.com/cloudflare/circl/releases) - [Commits](cloudflare/circl@v1.3.3...v1.3.7) --- updated-dependencies: - dependency-name: github.com/cloudflare/circl dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Fix mtu requirement bug There a bug with the MTU field in terraform where ExactlyOneOf requires Jumbo or MTU to be set, but the intent was to allow at most one of those fields to be set, but are optional. Add satellite features and add support to the user can direct which security groups are added to their workers Add volume clone resource and data source (#5019) * Add volume clone resource and data source * Docs improved * power-go-client update in go.mod * feedback resolved * testcase func renamed * Resolved conflicts * feedback resolved renamed targetReplicationEnabled to replicationEnabled * Alphabetically sorted documentation attributes. * Improved minor naming convention & docs * Added some use case scenarios * Feedback resolved * Fix: Document formatting * Making use of the correct cloudInstanceId constant add resource instance sample config code part in the website doc listing all the connection for transit gateway over the pagination set SCC Update Jan 2024 (#5032) * init commit * Updates: - Added ENV_VARS needed to run ACC tests - Added a validator for the resource scc_instance_settings - Allow the ability for the scc_instance_settings to be imported - Completed the functional testing for resource ibm_scc_instance_settings - Added documentation for scc_instance - Added documentation for scc_instance_settings * adding updates to scc_instance docs * Changed the format of the website documentation * Change the details of the NOTE for scc_instance * Modifying scc test runs --------- Co-authored-by: Timothy-Yao <[email protected]> fix(vpc-routing-table): support removing of advertise routes and accept routes from array (#5039) * fix(vpc-routing-table): support removing of advertise routes and accept routes from arrays * updated test cases Projects issue #2672 - "Terraform sees inputs as changed when they have not" (#5042) * typo * typo * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * add wait for create and delete * add wait for create and delete * . * fix integration tests * fix integration tests * fix integration tests * fix integration tests * fix integration tests * regenerate * new code generation * new generated code * point to latest sdk * point to latest sdk * adjust test cases and doc * adjust samples * update to latest node sdk * merge with master * Update project_instance.html.markdown Fixing formatting issues * Update project_event_notification.html.markdown * Update README.md * Update variables.tf * Update project_event_notification.html.markdown * new version * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * terraform * feat: projects graph fragment pattern * align with origin/master * update test cases * restore README.md and versions.tf to master branch * remove duplicated entries * feat: graph fragment pattern poc * feat: restore definition blocks * regenerate the terraform code based on latest YAML * regenerate the terraform code based on latest YAML * regenerate the terraform code based on latest YAML * update doc * update doc * fix test cases * commit generated code * commit fix * commit fix * test cases * test cases * test cases * test cases * updates * upgrade to latest go sdk * update generated doc * update generated code * updated secrets baseline * update code to latest API changes * secrets * fix secrets * update samples with real values * update secrets * update secrets * update generated code to the latest API definition * update secrets baseline * merge with master * merge with master * merge with master * update code to the latest SDK level * address some review comments * address PR comments * address PR comments * update to latest sdk level * update to latest sdk level * update to latest sdk level * merge with master * update to the latest sdk level * format test case * regenerate the code with generator fixes * regenerate code * fix "inputs" and "settings" properties * regenerated doc and samples * fix samples * terraform * terraform * terraform * terraform * terraform * fix marshal strings * secrets * resolve conflicts --------- Co-authored-by: knee-na <[email protected]> Co-authored-by: Mark-Kulube <[email protected]> added a nil check on data source of bm servers Soft remove the datasources for secretManager v1 Schematics agent related fixes for GA (#5041) * increased timeout of agent deploy to 30min * Fix for agent and policy tags crash issue * removed x-feature-agents header references * added refresh_token for agent update * Updated agent resource doc with latest * enchanced agent update * Modified wait logic of agent deploy with latest agent status code * Minor edits to agent testcases support parameters for resource instance datasource remove forcenew from workerpool fields in cluster resource and added ApplyOnce (#4955) * remove forcenew from workerpool fields in cluster resource and added ApplyOnce * doc update * remove ForceNew lines, added ApplyOnce in vpc taints, Doc Update * import on create * log and error change * remove some deprecated code, updated create and update funcs. * id vars * tains not availble in read, no point making it computed * remove irrelevant update for worker_num in doc related to applyonce * review comments and vpc testcase * test import_on_create in classic * Update ibm/service/kubernetes/resource_ibm_container_vpc_worker_pool.go Co-authored-by: Zoltán Illés <[email protected]> * Update ibm/service/kubernetes/resource_ibm_container_vpc_worker_pool.go Co-authored-by: Zoltán Illés <[email protected]> * review * Update ibm/service/kubernetes/resource_ibm_container_vpc_cluster.go Co-authored-by: Zoltán Illés <[email protected]> * Update ibm/service/kubernetes/resource_ibm_container_vpc_cluster.go Co-authored-by: Zoltán Illés <[email protected]> * Update ibm/service/kubernetes/resource_ibm_container_vpc_cluster.go Co-authored-by: Zoltán Illés <[email protected]> * labels apply once in classic cluster resource * more test steps for classic wp * more test steps for vpc wp * kms in create cluster * comment header year * conflict fix * call update after create terrafrom to apply rest of the modifications * update after cluster creation * remove enable kms from create, as it is present in update too --------- Co-authored-by: Zoltán Illés <[email protected]> Update terraform example (#5050) * Update powervs terraform example with workspace creation * Update workspace creation construct * Update datacenter location comment * Remove explicit depends on text from example SM fixes (#5045) * SC addition * SC addition * SC addition * update function updated * SC unit tests added * SC unit tests added * d * tests fixes * tests fixes * update sdk * .secrets.baseline update * .secrets.baseline update * .secrets.baseline update * Update sm_service_credentials_secret_metadata.html.markdown * bugs fixes * bugs fixes * bugs fixes * docs bugs fixes * preferred_chain added for public cert lets encrypt configuration * support for creating secret version for username password & version_custom_metadata for all * support for creating secret version for username password & version_custom_metadata for all * support for creating secret version for username password & version_custom_metadata for all * support for creating secret version for username password & version_custom_metadata for all * support for creating secret version for username password & version_custom_metadata for all * support for creating secret version for username password & version_custom_metadata for all --------- Co-authored-by: Yonathan-Yellin <[email protected]> Co-authored-by: Avi Ribchinsky <[email protected]> Co-authored-by: Tatyana <[email protected]> Co-authored-by: Idan Adar <[email protected]> Bump up version to 1.62.0-beta0 add isWaitForPIInstanceShutoff modify update vopt

…pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#89)

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#140)

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#286)

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#106)

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#346)

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#274)

Alex Hemard added 4 commits November 16, 2023 14:05

feat(Cloud Databases): Database user password complexity validation

b36bc46

update secrets

5c72c00

update tests to conform to new complexity rules

18571fe

Merge remote-tracking branch 'upstream/master' into cdb-password-comp…

6ac093a

…lexity

alexhemard force-pushed the cdb-password-complexity branch from 12a16ad to 6ac093a Compare November 17, 2023 11:57

Alex Hemard and others added 6 commits November 17, 2023 08:18

support go 1.18

352af86

fix password validation edgecases

0e4b074

add test case for password beginning with special char

eef99ef

fix regexp

e0e84a0

refactor of db users

972df9c

Merge branch 'master' into cdb-password-complexity

4b28142

alexhemard commented Nov 22, 2023

View reviewed changes

Alex Hemard and others added 2 commits November 22, 2023 13:16

remove debug logs

3321d00

Merge branch 'master' into cdb-password-complexity

492bf2b

alexhemard mentioned this pull request Nov 27, 2023

Error message for adminpassword (ibm_database) length requirement is not accurate. #4951

Closed

omaraibrahim reviewed Nov 27, 2023

View reviewed changes

omaraibrahim approved these changes Nov 27, 2023

View reviewed changes

obai-1 reviewed Nov 27, 2023

View reviewed changes

Alex Hemard added 4 commits November 28, 2023 19:06

delete and create ops manager users when updating

69b10b8

Merge remote-tracking branch 'origin/cdb-password-complexity' into cd…

8cbb46c

…b-password-complexity

validate adminpassword, update docs

e1584ed

update secrets

0e4cceb

obai-1 merged commit dc6d1a9 into IBM-Cloud:master Nov 29, 2023
2 checks passed

ocofaigh pushed a commit to terraform-ibm-modules/terraform-ibm-icd-etcd that referenced this pull request Jan 30, 2024

fix: updated required IBM provider version to >= 1.61.0, <2.0.0 to …

a6d1eb2

…pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#89)

ocofaigh pushed a commit to terraform-ibm-modules/terraform-ibm-icd-edb that referenced this pull request Jan 31, 2024

fix(deps): updated required IBM provider version to `>= 1.61.0, <2.0.…

8fa6a83

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#140)

ocofaigh pushed a commit to terraform-ibm-modules/terraform-ibm-icd-mongodb that referenced this pull request Jan 31, 2024

fix(deps): updated required IBM provider version to `>= 1.61.0, <2.0.…

54378a2

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#286)

ocofaigh pushed a commit to terraform-ibm-modules/terraform-ibm-icd-redis that referenced this pull request Jan 31, 2024

fix(deps): updated required IBM provider version to `>= 1.61.0, <2.0.…

8126b64

…0` to pickup fix for [this](IBM-Cloud/terraform-provider-ibm#4931) provider issue (#274)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(Cloud Databases): Database user password complexity validation #4931

feat(Cloud Databases): Database user password complexity validation #4931

alexhemard commented Nov 16, 2023 •

edited

Loading

alexhemard Nov 22, 2023

omaraibrahim Nov 27, 2023

alexhemard Nov 27, 2023 •

edited

Loading

omaraibrahim Nov 27, 2023

omaraibrahim left a comment

obai-1 Nov 27, 2023

ocofaigh commented Dec 5, 2023

feat(Cloud Databases): Database user password complexity validation #4931

feat(Cloud Databases): Database user password complexity validation #4931

Conversation

alexhemard commented Nov 16, 2023 • edited Loading

Community Note

alexhemard Nov 22, 2023

Choose a reason for hiding this comment

omaraibrahim Nov 27, 2023

Choose a reason for hiding this comment

alexhemard Nov 27, 2023 • edited Loading

Choose a reason for hiding this comment

omaraibrahim Nov 27, 2023

Choose a reason for hiding this comment

omaraibrahim left a comment

Choose a reason for hiding this comment

obai-1 Nov 27, 2023

Choose a reason for hiding this comment

ocofaigh commented Dec 5, 2023

alexhemard commented Nov 16, 2023 •

edited

Loading

alexhemard Nov 27, 2023 •

edited

Loading