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.

Sign up for GitHub

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

Jump to bottom

Add experimental peadm::util::code_sync_status plan #221

Merged
merged 3 commits into from
Apr 6, 2022
Merged

Add experimental peadm::util::code_sync_status plan #221

merged 3 commits into from
Apr 6, 2022

Conversation

reidmv
Copy link
Contributor

@reidmv reidmv commented Oct 4, 2021

Just a draft to save some doodling. The JSON from the task is not the easiest to read, compared to output such as the following.

Starting: plan peadm::util::code_sync_status
Starting: task peadm::code_sync_status on pe-server-37cf31-0.us-west1-a.c.puppet-solutions-architects.internal
Finished: task peadm::code_sync_status with 0 failures in 4.49 sec
+--------------------------------+------+
|                Summary                |
+--------------------------------+------+
| Overall sync status            | true |
| production environment in sync | true |
+--------------------------------+------+
+-----------------------------------------------------------------------------------+---------+---------------------------------------------+
|                                                      Server sync status - production                                                      |
+-----------------------------------------------------------------------------------+---------+---------------------------------------------+
| Server                                                                            | In Sync | Commit                                      |
+-----------------------------------------------------------------------------------+---------+---------------------------------------------+
| pe-server-37cf31-0.us-west1-a.c.puppet-solutions-architects.internal              | true    | Commit content to file sync storage service |
| pe-compiler-37cf31-3.us-west1-a.c.puppet-solutions-architects.internal            | true    | Commit content to file sync storage service |
| pe-server-37cf31-1.us-west1-b.c.puppet-solutions-architects.internal              | true    | Commit content to file sync storage service |
| pe-compiler-37cf31-0.us-west1-a.c.puppet-solutions-architects.internal            | true    | Commit content to file sync storage service |
| pe-compiler-37cf31-2.us-west1-c.c.puppet-solutions-architects.internal            | true    | Commit content to file sync storage service |
| pe-compiler-37cf31-1.us-west1-b.c.puppet-solutions-architects.internal            | true    | Commit content to file sync storage service |
| pe-server-37cf31-0.us-west1-a.c.puppet-solutions-architects.internal-orchestrator | true    | Commit content to file sync storage service |
+-----------------------------------------------------------------------------------+---------+---------------------------------------------+
Finished: plan peadm::util::code_sync_status in 4.53 sec
"Done"

@mcka1n mcka1n marked this pull request as ready for review April 6, 2022 17:12
@mcka1n mcka1n requested a review from a team as a code owner April 6, 2022 17:12
@mcka1n
adds the @api private comment on top
d219d8a 
This will help us to identify that this file is not Public API ready.
@mcka1n mcka1n merged commit 06f848e into puppetlabs:main Apr 6, 2022
@reidmv reidmv changed the title Create sample plan to display code sync status Add peadm::util::code_sync_status plan Apr 13, 2022
@reidmv reidmv changed the title Add peadm::util::code_sync_status plan Experimental: add peadm::util::code_sync_status plan Apr 13, 2022
@reidmv reidmv changed the title Experimental: add peadm::util::code_sync_status plan Add experimental peadm::util::code_sync_status plan Apr 13, 2022
@reidmv reidmv added the feature label Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcka1n mcka1n mcka1n approved these changes

Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@reidmv @mcka1n