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

Request permissions to release configurationslicing #1536

Merged
merged 1 commit into from May 24, 2020

Conversation

guysoft
Copy link
Contributor

@guysoft guysoft commented May 23, 2020

Hey,

Description

I Have recently adopted the plugin, I have a new version ready to release but I am getting:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.8.2:deploy (default-deploy) on project configurationslicing: Failed to deploy artifacts: Could not transfer artifact org.jenkins-ci.plugins:configurationslicing:hpi:1.50 from/to maven.jenkins-ci.org (https://repo.jenkins-ci.org/releases/): Transfer failed for https://repo.jenkins-ci.org/releases/org/jenkins-ci/plugins/configurationslicing/1.50/configurationslicing-1.50.hpi 401 Unauthorized -> [Help 1]

Submitter checklist for changing permissions

Always

For a newly hosted plugin only

  • Add link to resolved HOSTING issue in description above

For a new permissions file only

When adding new uploaders (this includes newly created permissions files)

@guysoft

Reviewer checklist (not for requesters!)

  • Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
  • Check that the $pluginId Developers team has Admin permissions while granting the access.
  • In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
  • If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Hey,
I Have recently adopted the plugin, I have a new version ready to release but I am getting:

```
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.8.2:deploy (default-deploy) on project configurationslicing: Failed to deploy artifacts: Could not transfer artifact org.jenkins-ci.plugins:configurationslicing:hpi:1.50 from/to maven.jenkins-ci.org (https://repo.jenkins-ci.org/releases/): Transfer failed for https://repo.jenkins-ci.org/releases/org/jenkins-ci/plugins/configurationslicing/1.50/configurationslicing-1.50.hpi 401 Unauthorized -> [Help 1]
```
@guysoft guysoft requested a review from a team as a code owner May 23, 2020 13:49
@guysoft guysoft changed the title Reqest permissios to release configurationslicing Request permissions to release configurationslicing May 23, 2020
@halkeye
Copy link
Member

halkeye commented May 23, 2020

[2020-05-23T13:53:05.930Z] Exception in thread "main" java.lang.IllegalStateException: User name not known to Artifactory: guysoft

Have you logged into artifactory yet?

@guysoft
Copy link
Contributor Author

guysoft commented May 23, 2020

Yes, Just making sure this means I am in right?

Also I can see my username here: https://reports.jenkins.io/artifactory-ldap-users-report.json

Screenshot_20200523_214934

@halkeye
Copy link
Member

halkeye commented May 23, 2020

Yep, it syncs the list of people logged in every 15 minutes (believe), so it probably hadn't syned yet by the time the CI ran.

I don't have permissions to re-run it, if you have the ability to close, wait a few seconds and reopen this, it should re-run.

Otherwise wait for those with access

guysoft added a commit to guysoft/repository-permissions-updater that referenced this pull request May 23, 2020
@guysoft
Copy link
Contributor Author

guysoft commented May 23, 2020

Closing, see #1539

@guysoft guysoft closed this May 23, 2020
@halkeye
Copy link
Member

halkeye commented May 23, 2020

oh okay, close and reopen just meant click the button twice type thing, but that totally works :)

@guysoft guysoft reopened this May 23, 2020
@guysoft
Copy link
Contributor Author

guysoft commented May 23, 2020

Ah, ok, I thought you need a whole new PR.
Done, should build and work since the other PR did.

@guysoft
Copy link
Contributor Author

guysoft commented May 24, 2020

@halkeye Hey, its passing - is there anything else you need?

@halkeye
Copy link
Member

halkeye commented May 24, 2020

Oh I just randomly help with reviews. I don't have approval access.

@halkeye
Copy link
Member

halkeye commented May 24, 2020

I'll nudge though

@oleg-nenashev oleg-nenashev merged commit f65e710 into jenkins-infra:master May 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants