Details
-
Task
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
-
frank, tethys, christmas
Description
Requirements
- Read natively from any branch of a Git repository hosted on Github
- Read from any branch of a Git repository
- Must maintain compatibility with Jenkins 2.7.1 and above
Desired user experience
- User experience is described in
JENKINS-41966 - Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047
Notes
jglick said there is beginnings of this in the scm-api but no implementations.
Attachments
Issue Links
- duplicates
-
JENKINS-39355 API improvements based on real-world implementation and usage
-
- Closed
-
- is blocking
-
JENKINS-36291
Pipeline Editor
-
- Open
-
- relates to
-
-
- Closed
-
-
-
- Resolved
-
-
-
- Resolved
-
Activity
| Field | Original Value | New Value |
|---|---|---|
| Labels | blueocean |
jglick where's the duplicate? I can close this one out and point my other references to this one.
Related to JENKINS-33273; stephenconnolly is already building the API & implementation.
| Link |
This issue relates to |
| Link |
This issue duplicates |
| Resolution | Duplicate [ 3 ] | |
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Component/s | bitbucket-plugin [ 18755 ] | |
| Component/s | git-plugin [ 15543 ] | |
| Component/s | github-plugin [ 15896 ] | |
| Component/s | scm-api-plugin [ 18054 ] |
Re-opening as this is a tracking ticket for whatever needs to be done to fit the requirements described in this issue.
| Resolution | Duplicate [ 3 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] |
| Assignee | James Dumay [ jamesdumay ] | Stephen Connolly [ stephenconnolly ] |
| Sprint | frank [ 156 ] |
| Rank | Ranked lower |
| Epic Link | JENKINS-36291 [ 172300 ] |
| Sprint | frank [ 156 ] | frank, christmas [ 156, 176 ] |
| Sprint | frank, christmas [ 156, 176 ] | frank, tethys, christmas [ 156, 161, 176 ] |
The read part is there. IMO no write API should be offered; this is just a big can of worms.
| Sprint | frank, tethys, christmas [ 156, 161, 176 ] | frank, tethys, christmas, pannonian [ 156, 161, 176, 211 ] |
| Assignee | Stephen Connolly [ stephenconnolly ] | James Dumay [ jamesdumay ] |
| Assignee | James Dumay [ jamesdumay ] |
| Sprint | frank, tethys, christmas, pannonian [ 156, 161, 176, 211 ] | frank, tethys, christmas [ 156, 161, 176 ] |
| Sprint | frank, tethys, christmas [ 156, 161, 176 ] | frank, tethys, christmas, iapetus [ 156, 161, 176, 216 ] |
| Rank | Ranked higher |
| Sprint | frank, tethys, christmas, iapetus [ 156, 161, 176, 216 ] | frank, tethys, christmas, post-release [ 156, 161, 176, 181 ] |
| Rank | Ranked lower |
| Description |
For the [Blue Ocean Pipeline Editor|https://jenkins.io/blog/2016/09/19/blueocean-beta-declarative-pipeline-pipeline-editor/] we want to read the Jenkinsfile from the repository, let the user author it in the editor then commit it back to the repository. We need a new API and relevant implementations (Github and Bitbucket) that would allow Blue Ocean to read/write a file from the repository. [~jglick] said there is beginnings of this in the scm-api but no implementations. |
*Scope* * Github *Out of scope* * Git and Bitbucket support *Notes* For the [Blue Ocean Pipeline Editor|https://jenkins.io/blog/2016/09/19/blueocean-beta-declarative-pipeline-pipeline-editor/] we want to read the Jenkinsfile from the repository, let the user author it in the editor then commit it back to the repository. We need a new API and relevant implementations (Github and Bitbucket) that would allow Blue Ocean to read/write a file from the repository. [~jglick] said there is beginnings of this in the scm-api but no implementations. |
| Rank | Ranked higher |
| Link |
This issue relates to |
| Assignee | Stephen Connolly [ stephenconnolly ] |
stephenconnolly and kzantow can the two of you confirm that API to read from a repository is done and meets expectations. Then we can close this and focus on the ticket for write API separately. JENKINS-42080
| Summary | API to read/write a single file from/to the repository | API to read a single file from the repository |
| Description |
*Scope* * Github *Out of scope* * Git and Bitbucket support *Notes* For the [Blue Ocean Pipeline Editor|https://jenkins.io/blog/2016/09/19/blueocean-beta-declarative-pipeline-pipeline-editor/] we want to read the Jenkinsfile from the repository, let the user author it in the editor then commit it back to the repository. We need a new API and relevant implementations (Github and Bitbucket) that would allow Blue Ocean to read/write a file from the repository. [~jglick] said there is beginnings of this in the scm-api but no implementations. |
*Scope* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade *Notes* For the [Blue Ocean Pipeline Editor|https://jenkins.io/blog/2016/09/19/blueocean-beta-declarative-pipeline-pipeline-editor/] we want to read the Jenkinsfile from the repository, let the user author it in the editor then commit it back to the repository. We need a new API and relevant implementations (Github and Bitbucket) that would allow Blue Ocean to read/write a file from the repository. [~jglick] said there is beginnings of this in the scm-api but no implementations. |
| Description |
*Scope* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade *Notes* For the [Blue Ocean Pipeline Editor|https://jenkins.io/blog/2016/09/19/blueocean-beta-declarative-pipeline-pipeline-editor/] we want to read the Jenkinsfile from the repository, let the user author it in the editor then commit it back to the repository. We need a new API and relevant implementations (Github and Bitbucket) that would allow Blue Ocean to read/write a file from the repository. [~jglick] said there is beginnings of this in the scm-api but no implementations. |
*Scope* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade. Must maintain compatibility with Jenkins 2.7.1 and above. *Desired user experience* Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047 *Notes* [~jglick] said there is beginnings of this in the scm-api but no implementations. |
| Link | This issue is blocking JENKINS-36291 [ JENKINS-36291 ] |
| Link |
This issue relates to |
| Link |
This issue relates to |
| Description |
*Scope* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade. Must maintain compatibility with Jenkins 2.7.1 and above. *Desired user experience* Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047 *Notes* [~jglick] said there is beginnings of this in the scm-api but no implementations. |
*Scope* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade. Must maintain compatibility with Jenkins 2.7.1 and above. *Desired user experience* * User experience is described in * Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047 *Notes* [~jglick] said there is beginnings of this in the scm-api but no implementations. |
| Description |
*Scope* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade. Must maintain compatibility with Jenkins 2.7.1 and above. *Desired user experience* * User experience is described in * Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047 *Notes* [~jglick] said there is beginnings of this in the scm-api but no implementations. |
*Requirements* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade. Must maintain compatibility with Jenkins 2.7.1 and above. *Desired user experience* * User experience is described in * Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047 *Notes* [~jglick] said there is beginnings of this in the scm-api but no implementations. |
| Description |
*Requirements* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must not require a Jenkins core upgrade. Must maintain compatibility with Jenkins 2.7.1 and above. *Desired user experience* * User experience is described in * Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047 *Notes* [~jglick] said there is beginnings of this in the scm-api but no implementations. |
*Requirements* * Read natively from any branch of a Git repository hosted on Github * Read from any branch of a Git repository * Must maintain compatibility with Jenkins 2.7.1 and above *Desired user experience* * User experience is described in * Mockup: https://projects.invisionapp.com/share/Q2A110KM9#/screens/219088047 *Notes* [~jglick] said there is beginnings of this in the scm-api but no implementations. |
| Link |
This issue relates to |
| Sprint | frank, tethys, christmas, post-release [ 156, 161, 176, 181 ] | frank, tethys, christmas [ 156, 161, 176 ] |
| Rank | Ranked higher |
hrmpw currently it is missing returning the content sha to allow saving back to github - Vivek is looking if we can calculate this to make it unnecessary, otherwise we won't be able to use it for the purposes of load/save just yet.
jglick I probably worded that poorly - the sha is required at some point or else you won't be able to save back any changes.
In theory this can be calculated, but github don't specify the 11 secret herbs and spices for how it is calculated.
All I found was this: http://stackoverflow.com/questions/39873507/how-to-calculate-sha-from-a-string-with-the-github-api-in-python
which uses a string involving "blob" and length (wut? why?) - which implies they may change how it is calculated, and that is why they return it on content load.
So it may be required on load (hopefully not though).
I have opened to get GitHub API returned data (sha included), https://issues.jenkins-ci.org/browse/JENKINS-42270. We definitely need sha and other elements returned by GitHub should be pretty much pass thru, otherwise it defeats the purpose of consumers who are really interested in getting content data specific to GitHub.
For now I am using 'git' way of computing sha, its a hack for now as who knows git changes it in future, better I get it from upstream library/github directly:
DigestUtils.sha1Hex("blob " + data.length() + "\0" + data)
Ah right - yes it is a git standard, of course. Of course sha1 is topical this week...
In any case, this ticket can be closed, I consider it done (now have seen it working). There is a follow on for github specific fields/properties which isn't needed right now.
| Resolution | Fixed [ 1 ] | |
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
dupe