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-38815 add_replica plan updated, test matrix added #480

Merged
merged 14 commits into from
Aug 28, 2024

Conversation

AaronShannon
Copy link
Contributor

@AaronShannon AaronShannon commented Aug 20, 2024

Summary

add_replica plan updated with correct certname passed to update_classification plan, test matrix added

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

echo ::group::info:request
cat request.json || true; echo
echo ::endgroup::
fi
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add the missing \n in line 124? Otherwise the git diff will look a bit suboptimal when another line is added in the future.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New line added

@AaronShannon AaronShannon marked this pull request as ready for review August 27, 2024 15:30
@AaronShannon AaronShannon requested review from a team as code owners August 27, 2024 15:30
@AaronShannon
Copy link
Contributor Author

@AaronShannon
Copy link
Contributor Author

Copy link
Member

@ragingra ragingra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@AaronShannon AaronShannon merged commit 2488a4d into add_replica_supported Aug 28, 2024
4 checks passed
@AaronShannon AaronShannon deleted the PE-38815-add_replica-test-matrix branch August 28, 2024 12:24
server_b_host => $replica_avail_group_letter ? { 'B' => $replica_host, default => undef },
internal_compiler_a_pool_address => $replica_avail_group_letter ? { 'A' => $replica_host, default => undef },
internal_compiler_b_pool_address => $replica_avail_group_letter ? { 'B' => $replica_host, default => undef },
server_a_host => $replica_avail_group_letter ? { 'A' => $replica_target.peadm::certname(), default => undef },
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At the top this plan still has:

# This plan is still in development and currently considered experimental.
#
# @api private

is that still the case?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is being removed in the below PR:

#478

CoMfUcIoS added a commit that referenced this pull request Sep 12, 2024
* PE-38815 add_replica plan updated, test matrix added (#480)

* (PE-38818) Removing experimental and private from add_replica (#478)

* (PE-38818) Removing experimental and private from add_replica

* Updating Reference.md

---------

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

* (PE-38817) Adding docs for add replica (#477)

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

* chore: update test-add-replica workflow

- Remove unnecessary quotes from YAML keys and values
- Standardize single quotes for string values
- Improve readability by removing extra blank lines

* feat(workflows): add console password to test-add-replica-matrix job

- Included `console_password` parameter in the test-add-replica-matrix job.
- Utilizes `${{ secrets.CONSOLE_PASSWORD }}` for secure password management.

* (PE-38817) Text fixes for add_replica docs (#496)

* PE-39228 add_replica smoke test (#497)

* PE-39228 Smoke test created and added to workflow

* PE-39228 Workflow updated

* PE-39228 Spec updated, smoke test added to matrix

* PE-39228 Versions updated, push trigger removed from matrix

* PE-39228 whitespace fixed

* chore(workflow): update test-add-replica-matrix to use single version

- Removed version `2021.7.9` from the matrix, leaving only `2023.8.0`.

This change simplifies the testing matrix by focusing on the latest version.

* chore(workflow): update test-add-replica-matrix to use single version

- Changed `version` matrix to use an array with only `2023.8.0`.

This change simplifies the testing matrix by focusing on the latest version.

* docs: update REFERENCE.md to use asterisks for list items

- Changed list item markers from hyphens to asterisks for consistency.
- Ensured all list items in the document follow the same format.

* add_replica docs updated

---------

Co-authored-by: Neil Anderson <ragingra@users.noreply.github.com>
Co-authored-by: Neil Anderson <neil.anderson@perforce.com>
Co-authored-by: Aaron Shannon <aaron.shannon@puppet.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