-
Notifications
You must be signed in to change notification settings - Fork 20
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
Update Mango to Spark 2.4.3 #292
Conversation
Hi @akmorrow13 — Do you have write/merge privileges to this repo? I defended in early June and will not be a reliable PR master for this repo any longer. |
Congrats @jvivian! I do not believe I have merge/write privileges It says a review is required from reviewers with write access, and I am not authorized to merge this PR. |
@akmorrow13 — Thank you! It went well. I have permissions to grant you write privileges, just need Benedict to sign off. If this is time sensitive let me know and I'll go ahead and merge. |
Thanks @jvivian! It should be fine for the next couple days. |
@jvivian it still says that a review is required by a reviewer with write access. Is there anyone else I can request a review from that is active? |
Hi @akmorrow13 — I didn't realize write access wouldn't give you permissions to push without review. I just added you via an option to, "Specify people or teams allowed to dismiss pull request reviews." Let me know if that changes anything. |
Thanks @jvivian. I still don't think I have the right access. I still see the following: It states at the bottom that I am not authorized. |
Hi @akmorrow13 — I went ahead and approved. I also removed the requirement that status checks pass given the deprecation of Jenkins, so ideally next time you can just dismiss the review and will have the authority to merge. |
I've checked and you have "write" privileges to the repo so I'm not quite sure why only I'm authorized to merge. Will keep looking, in the mean time I merged. |
Thanks @jvivian ! Let me know if there is something on my end I have to do. |
I think I fixed it — push privileges was under a separate category for this "protected branch." Hopefully that resolves it, otherwise just ping me. |
@jvivian how was the quay.io build triggered? Does someone have to manually do that? If, so I could take that on in the future I just need the instructions. |
@akmorrow13 — I checked the tags and it wasn't pushed to Quay (which makes sense given that there's no associated Jenkins build anymore). We could try and get this repo onto Travis CI, or an easier option might be to just use this repo for version control and push the containers you care about to your personal Dockerhub. Unfortunately, I don't think anyone in the CGL is actively using, maintaining, or contributing to this repo. |
Thanks @jvivian. Is there anyway I could use the .dockercfg file to push one last time? I can move the project to a new quay repository in the future. However, I just cut a release that depends on the quay.io cgl Mango path. This way I could just 'make push' the Mango project. |
I'm trying to get it up on quay now, will keep you posted. |
Thank you so much @jvivian, I owe you all the beers. |
No problem! 🍻 https://quay.io/repository/ucsc_cgl/mango?tab=tags |
Thanks so much @jvivian! |
No description provided.