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

netapp_api_luns: Added missing Lun Informations #605

Closed
wants to merge 2 commits into from

Conversation

Bastian-Kuhn
Copy link
Contributor

If you have a Setup with Multiple Netapps and Thousands of Luns, it's currently not possible to figure out to where a Lun belongs.

With this small Change, this information is now part of the Check Output.

I have read the CLA Document and I hereby sign the CLA or my organization already has a signed CLA.

@github-actions
Copy link

github-actions bot commented Jun 29, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@BenediktSeidl
Copy link
Contributor

@Bastian-Kuhn I fear the "CLA Assistant" did not recognize your first message. I've fixed the tests and added a werk, so if you get past the "CLA Assistant" we should be able to merge your change. Sorry about the inconvenience, and thanks for your contribution.

@BenediktSeidl
Copy link
Contributor

recheck

@BenediktSeidl
Copy link
Contributor

@Bastian-Kuhn we would be ready from our side to merge this, you just have to "sign" the CLA

@Bastian-Kuhn
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA or my organization already has a signed CLA.

@BenediktSeidl
Copy link
Contributor

recheck

@CheckmkCI CheckmkCI closed this in 24f2d55 Nov 14, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Nov 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants