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

Enable CD #43

Merged
merged 5 commits into from
Apr 9, 2024
Merged

Enable CD #43

merged 5 commits into from
Apr 9, 2024

Conversation

bzzitsme
Copy link

@bzzitsme bzzitsme commented Apr 8, 2024

Testing done

Submitter checklist

@bzzitsme
Copy link
Author

bzzitsme commented Apr 8, 2024

@Vlatombe could you please review this PR?

Copy link
Member

@Vlatombe Vlatombe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bzzitsme bzzitsme requested a review from Vlatombe April 8, 2024 15:30
pom.xml Outdated Show resolved Hide resolved
@bzzitsme
Copy link
Author

bzzitsme commented Apr 8, 2024

It should be OK now, if I didn't miss anything. Added: cd.yaml, updated maven.config and pom.xml
all the other things were in the plugin already (was following: https://www.jenkins.io/doc/developer/publishing/releasing-cd/)

@bzzitsme bzzitsme requested a review from Vlatombe April 8, 2024 15:42
@Vlatombe
Copy link
Member

Vlatombe commented Apr 8, 2024

Still missing the dependabot entry

- package-ecosystem: github-actions
  directory: /
  schedule:
    interval: monthly

@bzzitsme
Copy link
Author

bzzitsme commented Apr 8, 2024

Sorry for that, didn't notice, I got confused with https://github.com/jenkinsci/kubernetes-credentials-plugin/blob/master/.github/dependabot.yml#L3-L8 (it's for maven, but not github-actions)
will add it now

@bzzitsme
Copy link
Author

bzzitsme commented Apr 8, 2024

Added in 780b751

@Vlatombe
Copy link
Member

Vlatombe commented Apr 8, 2024

Need jenkins-infra/repository-permissions-updater#3875 merged first, and this repository updated with the corresponding secrets for deployment (which happens automatically some time after)

@bzzitsme
Copy link
Author

bzzitsme commented Apr 8, 2024

Thanks a lot for the review, @Vlatombe

@bzzitsme
Copy link
Author

bzzitsme commented Apr 9, 2024

PR in RPU was merged, I think we can merge this PR now

@Vlatombe
Copy link
Member

Vlatombe commented Apr 9, 2024

@maxlaverse FYI, and can you check https://www.jenkins.io/doc/developer/publishing/releasing-cd/#troubleshooting ? I don't have administrative right on this repo (only write) so can't check it.

@maxlaverse
Copy link

hi @Vlatombe ! I read the troubleshooting page. What do I need to troubleshoot ? It's meant for artifact upload errors, that would likely fail because of missing secrets, but those would be set automatically on this repository ?

@Vlatombe
Copy link
Member

Vlatombe commented Apr 9, 2024

It's only to doublecheck that the CD setup happened properly and that the expected secrets are set in the repository. Otherwise we can just blindly merge and see whether the deployment happens as expected.

@maxlaverse
Copy link

Looks like the environment variables are here:
image

GitHub actions are disabled however. Does it need more permission than that ? (couldn't see it on https://www.jenkins.io/doc/developer/publishing/releasing-cd/)
image

@Vlatombe
Copy link
Member

Vlatombe commented Apr 9, 2024

I'm merging, then.

@Vlatombe Vlatombe merged commit 44a109d into jenkinsci:master Apr 9, 2024
13 checks passed
@lemeurherve
Copy link
Member

@maxlaverse I think you'll have to tick the last option and add jenkins-infra/github-reusable-workflows to the selected non-jenkinsci actions as currently the GHA is failing:

jenkins-infra/github-reusable-workflows/.github/workflows/maven-cd.yml@v1 is not allowed to be used in jenkinsci/kubernetes-credentials-plugin. Reusable workflows in this workflow must be: within a repository owned by jenkinsci.

@maxlaverse
Copy link

I fixed the permissions issue @lemeurherve . Build is running:
image

@maxlaverse
Copy link

Could it be that a prefix is missing for the tag ? It starts with a -
image

@lemeurherve
Copy link
Member

It seems so yes, no idea how to resolve this though, sorry.

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

Successfully merging this pull request may close these issues.

4 participants