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

Added Playbooks and Analytic Rules for Pure Storage Solutions #10379

Merged
merged 11 commits into from May 7, 2024

Conversation

skeerthivasan
Copy link
Contributor

Required items, please complete

Change(s):
Addition of playbooks and analytic rules for Pure Storage Solutions

Reason for Change(s):
Addition of playbooks and analytic rules for Pure Storage Solutions

Version Updated:
Yes

Testing Completed:
Yes

Checked that the validations are passing and have addressed any issues that are present:
Yes

@skeerthivasan skeerthivasan requested review from a team as code owners April 24, 2024 21:50
@v-prasadboke v-prasadboke self-assigned this Apr 25, 2024
@v-prasadboke v-prasadboke added the Solution Solution specialty review needed label Apr 25, 2024
@skeerthivasan
Copy link
Contributor Author

@v-prasadboke , Thanks for looking into this, Let me know if there are any actions required from my end.

@v-prasadboke
Copy link
Contributor

@v-prasadboke , Thanks for looking into this, Let me know if there are any actions required from my end.

Hello @skeerthivasan, Please fill up the metadata details in all the playbooks

@skeerthivasan
Copy link
Contributor Author

@v-prasadboke , Thanks for looking into this, Let me know if there are any actions required from my end.

Hello @skeerthivasan, Please fill up the metadata details in all the playbooks

Hi @v-prasadboke , Thanks for your inputs, I've added requested details, please check and proceed.

@v-prasadboke
Copy link
Contributor

Thanks @skeerthivasan, For committing the required changes

@skeerthivasan
Copy link
Contributor Author

Thanks @skeerthivasan, For committing the required changes

@v-prasadboke , if its all fine, please proceed with merging. We need to push the changes into solutions package asap.

@skeerthivasan
Copy link
Contributor Author

Thanks @skeerthivasan, For committing the required changes

@v-prasadboke , if its all fine, please proceed with merging. We need to push the changes into solutions package asap.

@v-prasadboke , did you get chance to look at this ? please help us in expediting, thanks

@skeerthivasan
Copy link
Contributor Author

@v-prasadboke , I have fixed file issue and repushed, checks are passing now. Please check and proceed

@skeerthivasan
Copy link
Contributor Author

@v-prasadboke , I have fixed file issue and repushed, checks are passing now. Please check and proceed

@v-prasadboke , can you please provide an update on this ? Thanks for your understanding.

@v-prasadboke
Copy link
Contributor

@v-prasadboke , I have fixed file issue and repushed, checks are passing now. Please check and proceed

@v-prasadboke , can you please provide an update on this ? Thanks for your understanding.

Hello @skeerthivasan, There's something wrong with playbooks. After deploying the solution playbooks cannot be seen in the automation blade.

I'm finding the issue first

@skeerthivasan
Copy link
Contributor Author

@v-prasadboke , I have fixed file issue and repushed, checks are passing now. Please check and proceed

@v-prasadboke , can you please provide an update on this ? Thanks for your understanding.

Hello @skeerthivasan, There's something wrong with playbooks. After deploying the solution playbooks cannot be seen in the automation blade.

I'm finding the issue first

@v-prasadboke , thanks for the update, I haven't seen any errors while packaging. Its strange to have this issue.
Let me know if you have any further pointers to proceed ? If required, I can join on a call to assist, let me know.

@skeerthivasan
Copy link
Contributor Author

@v-prasadboke , I have fixed file issue and repushed, checks are passing now. Please check and proceed

@v-prasadboke , can you please provide an update on this ? Thanks for your understanding.

Hello @skeerthivasan, There's something wrong with playbooks. After deploying the solution playbooks cannot be seen in the automation blade.
I'm finding the issue first

@v-prasadboke , thanks for the update, I haven't seen any errors while packaging. Its strange to have this issue. Let me know if you have any further pointers to proceed ? If required, I can join on a call to assist, let me know.

@v-prasadboke , I was able to figure out the template issue and fixed the same. Also tested using preview links, its working fine. Please check and proceed with merge. Also solution offer certification(pushed from partner page) is pending for manual validation. If that comes to your review, please push the same or let relevant team to expedite. Thanks in advance.

@skeerthivasan
Copy link
Contributor Author

@v-prasadboke , Any updates on this please

v-rusraut
v-rusraut previously approved these changes May 7, 2024
@v-prasadboke v-prasadboke merged commit cdcea0d into Azure:master May 7, 2024
33 checks passed
@skeerthivasan
Copy link
Contributor Author

@v-prasadboke @v-rusraut , Thanks for merging PR.
Please take care of Solution offer review in Marketplace as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Solution Solution specialty review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants