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

(PE-38814) add_compiler - Making primary_postgresql_host and avail_group_letter optional #468

Merged
merged 4 commits into from
Aug 12, 2024

Conversation

ragingra
Copy link
Member

@ragingra ragingra commented Jul 29, 2024

Summary

Making the following parameters optional:

  • primary_postgresql_host, if not provided will be determined through get_peadm_config
  • avail_group_letter, is defaulting to A

Additional Context

Add any additional context about the problem here.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.

Changes include test coverage?

  • Yes
  • Not needed

Have you updated the documentation?

  • Yes, I've updated the appropriate docs
  • Not needed

…oup_letter optional

primary_postgresql_host, if not provided will be determined through get_peadm_config
avail_group_letter, is defaulting to A
@ragingra ragingra marked this pull request as ready for review July 30, 2024 07:22
@ragingra ragingra requested review from a team as code owners July 30, 2024 07:22
plans/add_compiler.pp Outdated Show resolved Hide resolved
@ragingra ragingra marked this pull request as draft August 5, 2024 14:24
Removing optional from avail_group_letter as not required with enum default value
@ragingra ragingra marked this pull request as ready for review August 8, 2024 08:56
@ragingra ragingra merged commit e610a2a into add_compiler_supported Aug 12, 2024
5 checks passed
@ragingra ragingra deleted the PE-38814 branch August 12, 2024 08:42
CoMfUcIoS pushed a commit that referenced this pull request Sep 2, 2024
…oup_letter optional (#468)

* (PE-38814) add_compiler - Making primary_postgresql_host and avail_group_letter optional

primary_postgresql_host, if not provided will be determined through get_peadm_config
avail_group_letter, is defaulting to A

* Updating spec tests for add compiler

Removing optional from avail_group_letter as not required with enum default value

* Updating reference.md

* Fixing linting issues

---------

Co-authored-by: Neil Anderson <neil.anderson@perforce.com>
CoMfUcIoS pushed a commit that referenced this pull request Sep 9, 2024
…oup_letter optional (#468)

* (PE-38814) add_compiler - Making primary_postgresql_host and avail_group_letter optional

primary_postgresql_host, if not provided will be determined through get_peadm_config
avail_group_letter, is defaulting to A

* Updating spec tests for add compiler

Removing optional from avail_group_letter as not required with enum default value

* Updating reference.md

* Fixing linting issues

---------

Co-authored-by: Neil Anderson <neil.anderson@perforce.com>
CoMfUcIoS added a commit that referenced this pull request Sep 12, 2024
* (PE-37236) Remove experimental status of add_compiler (#428)

* (PE-37233) Adding add_compiler to test matrix (#434)

Updated add_compiler spec to check compiler has been added and puppet run before running add compiler
Added test matrix for add compiler

* (PE-38814) add_compiler - Making primary_postgresql_host and avail_group_letter optional (#468)

* (PE-38814) add_compiler - Making primary_postgresql_host and avail_group_letter optional

primary_postgresql_host, if not provided will be determined through get_peadm_config
avail_group_letter, is defaulting to A

* Updating spec tests for add compiler

Removing optional from avail_group_letter as not required with enum default value

* Updating reference.md

* Fixing linting issues

---------

Co-authored-by: Neil Anderson <neil.anderson@perforce.com>

* (PE-37235) Verify and update documentation on add_compiler (#429)

* PE-37235 Verify and update documentation on add_compiler if needed and its experimental status

* @cathal41
PE-37235 Verify and update documentation on add_compiler - wording updates

* Update plans/add_compiler.pp

Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>

* Update documentation/add_compiler.md

Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>

* Update documentation/add_compiler.md

Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>

* Update documentation/add_compiler.md

Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>

* Update documentation/add_compiler.md

Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>

* Update documentation/add_compiler.md

Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>

* Updating add compiler docs to show optional params

* Updating docs with text corrections

---------

Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>
Co-authored-by: Neil Anderson <neil.anderson@perforce.com>

* chore(workflows): simplify YAML syntax and improve readability

- Removed unnecessary quotes around strings in YAML files.
- Consolidated list items to single-line format where applicable.
- Ensured consistent naming conventions for job steps.
- Improved readability by removing extra blank lines.

* fix(workflow): add console_password to test-add-compiler workflows

- Added `console_password` parameter to the `test-add-compiler-matrix.yml` and `test-add-compiler.yaml` workflows.
- This ensures the console password is passed correctly during the workflow execution.

* chore: update test-add-compiler-matrix workflow

- Remove centos-7 from the image matrix in test-add-compiler-matrix.yml
- Retain only almalinux-cloud/almalinux-8 in the image matrix

* feat(workflow): add --stream option to bolt plan run command

Added the --stream option to the bolt plan run command in the test-add-compiler GitHub workflow. This change ensures that the output is streamed in real-time.

* chore(workflow): update compiler versions in test-add-compiler-matrix.yml

Updated the compiler versions in the test-add-compiler-matrix.yml GitHub workflow from 2021.7.7 and 2023.6.0 to 2021.7.9 and 2023.8.0.

---------

Co-authored-by: Jo Lillie <jo.lillie@puppet.com>
Co-authored-by: Neil Anderson <ragingra@users.noreply.github.com>
Co-authored-by: Neil Anderson <neil.anderson@perforce.com>
Co-authored-by: cathal41 <62099613+cathal41@users.noreply.github.com>
Co-authored-by: Dimitri Tischenko <1586813+timidri@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants