Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

[associated vote ended on 2022-12-16] Removal from Ansible 9: cyberark.pas #168

Closed
gotmax23 opened this issue Nov 30, 2022 · 14 comments
Closed

Comments

@gotmax23
Copy link
Contributor

Summary

I'm filing this issue in accordance with Removal from Ansible § Collections not satisfying the Collection Requirements.

On 27 October (over four weeks ago), I opened an issue (cyberark/ansible-security-automation-collection#46) against cyberark.pas's repository reporting their violation of the release tagging requirements. There has been no acknowledgment of the issue, and it hasn't been resolved, so it's eligible for removal from Ansible 9.

Two SC members need to confirm that this collection is indeed violating the requirements. After that, I'll give about a week long grace period before starting a vote on whether we should remove this collection from Ansible 9.

@mariolenz
Copy link
Contributor

At the time of writing this comment I don't see they tag their releases.

So I agree / confirm that this collection is indeed violating the requirements.

@felixfontein
Copy link
Contributor

I can also confirm that they do not tag their releases and thus violate the requirements. Also they have been informed about this more than a month ago, and there has been zero feedback so far.

@gotmax23
Copy link
Contributor Author

gotmax23 commented Dec 8, 2022

I've started a vote: #172

@gotmax23 gotmax23 changed the title Removal from Ansible 9: cyberark.pas [Vote ends on 2022-12-16] Removal from Ansible 9: cyberark.pas Dec 8, 2022
@gotmax23 gotmax23 added the active-vote These are currently active votes label Dec 8, 2022
@mariolenz
Copy link
Contributor

mariolenz commented Dec 16, 2022

I counted votes:
7 x +1 from SC (gotmax23 felixfontein russoz mariolenz briantist markuman ssbarnea)
2 x +1 from community (oraNod cybette)
edit: 1 x -1 from community (szh)

Can someone from the Steering Committee confirm this?

@jtuttle
Copy link

jtuttle commented Dec 16, 2022

Hi there, I'm the manager of the CyberArk Community & Integrations team at CyberArk. We are responsible for most of the OSS repositories but unfortunately this one doesn't fall under our umbrella. This lapse has just been brought to my attention. I will try to get the appropriate person / team working on it ASAP. Sorry for the delay.

@jtuttle
Copy link

jtuttle commented Dec 16, 2022

Update: I reached out to the person that manages @cyberark-bizdev directly and he said he will try to address it in the next few days, which I reckon will be next week sometime since it's Friday. I'll be on vacation next week but will try to hop on at some point to make sure this has been resolved.

@mariolenz mariolenz changed the title [Vote ends on 2022-12-16] Removal from Ansible 9: cyberark.pas [Vote ended on 2022-12-16] Removal from Ansible 9: cyberark.pas Dec 17, 2022
@mariolenz mariolenz removed the active-vote These are currently active votes label Dec 17, 2022
@felixfontein
Copy link
Contributor

I counted votes: 7 x +1 from SC (gotmax23 felixfontein russoz mariolenz briantist markuman ssbarnea) 2 x +1 from community (oraNod cybette) edit: 1 x -1 from community (szh)

Can someone from the Steering Committee confirm this?

I can confirm this count.

@felixfontein
Copy link
Contributor

We talked a bit (informally) on this on IRC/Matrix, and we (well, the folks involved in the discussion there) are not really sure how to proceed with this. While we're happy that there is now some reaction, it is really late, considering that cyberark/ansible-security-automation-collection#46 was created almost two months ago, and an earlier issue (cyberark/ansible-security-automation-collection#44) from May has not received any feedback so far. We'll probably discuss this again in January once most folks are back from possible vacations.

@gotmax23
Copy link
Contributor Author

Yeah, it's unfortunate that there was no response for two months and it only happened after we already agreed to remove the collection from the ansible package. I haven't announced the removal of this collection yet, but I guess we'll need another vote to keep this collection in the Ansible package if the issue is fixed. In any case, I tagged this for discussion in the next community meeting.

@gotmax23 gotmax23 added discussion_topic next_meeting Topics that needs to be discussed in the next Community Meeting labels Dec 18, 2022
@mhyclak-silex
Copy link

This topic was just brought to my attention. I can say that we're using essentially a fork of cyberark.pas for one of our customers due to the inattention the modules received. There were some helpful patches in the Issues on the github repo that we pulled in and incorporated that allowed us to continue working. I would have to investigate from a contractual/legal standpoint what (if anything) we can share back to the community, but one of my teammates wrote some tests around the module as well. It's a tough one to do since you need a full-blown implementation to validate. I'm not sure if I'll be able to participate in the next meeting, but at least wanted to drop this comment for consideration.

@felixfontein
Copy link
Contributor

We talked about this a bit at today's community meeting. We decided to wait a few more days (until ~mid January) to see what happens with the collection, and then potentially start another vote whether to delay / cancel the removal or not.

@jtuttle
Copy link

jtuttle commented Jan 4, 2023

@felixfontein Thanks, I'm still trying to get the folks in charge of this project to fix it.

@infamousjoeg
Copy link

Sorry this took so long. I went ahead and added a tag and release to match the version in Galaxy for the collection: https://github.com/cyberark/ansible-security-automation-collection/tree/1.0.14

@mariolenz mariolenz removed the next_meeting Topics that needs to be discussed in the next Community Meeting label Jan 26, 2023
@mariolenz
Copy link
Contributor

Closing because we decided to cancel the removal process in #180.

@felixfontein felixfontein changed the title [Vote ended on 2022-12-16] Removal from Ansible 9: cyberark.pas [associated vote ended on 2022-12-16] Removal from Ansible 9: cyberark.pas Mar 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Status: Resolved
Development

No branches or pull requests

6 participants