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

CI Pipeline interface #74

Closed
nkprince007 opened this issue Dec 19, 2017 · 13 comments
Closed

CI Pipeline interface #74

nkprince007 opened this issue Dec 19, 2017 · 13 comments
Assignees
Milestone

Comments

@nkprince007
Copy link
Contributor

nkprince007 commented Dec 19, 2017

NOTICE: Description fully cannibalized by @zaquestion
Still debating if such a thing should be in lab, but could always move it into its own tool and be called by lab.
Support

  • Running jobs
  • Viewing jobs status (live)
  • Viewing environments
    • Include re-deploying jobs

Also add lab ci lint command
Ref: https://docs.gitlab.com/ce/api/lint.html

@zaquestion
Copy link
Owner

Also started working on a ci sub command. We could support this through lab ci validate

@zaquestion zaquestion changed the title Support validate-ci to check whether the CI configuration is valid or not Support lab ci lint to check whether the CI configuration is valid or not Dec 19, 2017
@zaquestion
Copy link
Owner

a_start
From: https://gitlab.com/zaquestion/test/pipelines/15273642

Using https://github.com/jroimartin/gocui

@nkprince007
Copy link
Contributor Author

That looks cool, can we maybe have some colours and arrows?

@zaquestion zaquestion self-assigned this Dec 20, 2017
@zaquestion zaquestion changed the title Support lab ci lint to check whether the CI configuration is valid or not CI Pipeline interface Dec 20, 2017
Repository owner deleted a comment from nkprince007 Dec 20, 2017
@zaquestion
Copy link
Owner

Still static, but we're getting there!
lab_ci

From: https://gitlab.com/zaquestion/test/pipelines/15273642

@zaquestion
Copy link
Owner

Put this up in an effort to support lab ci lint. xanzy/go-gitlab#293

I've already written a working implementation with those changes. Once the API is exposed should be able to bring in the functionality in short order.

@zaquestion
Copy link
Owner

Also relying on these changes for the pipeline interface: jroimartin/gocui#130

@zaquestion
Copy link
Owner

zaquestion commented Feb 20, 2018

Have mostly replicated the current progress in https://github.com/rivo/tview which has been nicer to work with overall.
tview_ci_pipeline

  • fix the rendering overall (notice the extra lines on the right most side)
  • test live updating (implemented but not yet useful)
  • screen doesn't clear when resizing (or between all updates?)
  • Long term solution for drawing the lines (currently have tview hacked up to expose the screen to me)

@zaquestion
Copy link
Owner

Latest look
pipeline

zaquestion added a commit that referenced this issue Apr 23, 2018
[#74] Add (beta) CI Pipeline interface: live view-only
@zaquestion
Copy link
Owner

The above has been released in 0.11.0 under lab ci view as a beta feature.

Also lab ci trace was release in 0.11.0 for viewing ci job logs in realtime!

@twouters
Copy link

twouters commented Apr 23, 2018

Is there a way to close the view (except for Ctrl-c)?
Would be nice if we can use Esc and/or q to close the view.

zaquestion added a commit that referenced this issue Apr 23, 2018
[#74] support exiting `ci view` with ESC and "q"
@zaquestion
Copy link
Owner

@twouters Thanks for the feedback, I've added support for exiting with q and esc in 0.11.1.

@zaquestion
Copy link
Owner

Put lunixbochs/vtclean#7 up to support cleaning gitlab's trace file response. Hopefully that gets merged in and we can use it directly, but I'll give it a week while I get the remaining changes together in lab. Code for running/retrying jobs has been written and seems to function (pending tests). The vtclean change is to support viewing trace logs in the pipeline interface, which is mostly coded up, but has some formating issues due to the escape sequences returned in the API

@zaquestion zaquestion added this to the 0.12.0 milestone Jul 13, 2018
@zaquestion
Copy link
Owner

Going to call this good as of #175, any future bugs or features will be tracked in new issues.

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

No branches or pull requests

3 participants