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

feat(provider/kubernetes): runJob logs #4501

Merged
merged 1 commit into from
Nov 21, 2017

Conversation

ethanfrogers
Copy link
Contributor

add support for displaying logs for runJob

Clouddriver PR: spinnaker/clouddriver#2164

@danielpeach @lwander PTAL!

spinnaker/spinnaker#1961
spinnaker/spinnaker#1832

@ethanfrogers ethanfrogers force-pushed the runjob-logs branch 2 times, most recently from 6297d18 to 6b40e02 Compare November 20, 2017 23:16
@ethanfrogers
Copy link
Contributor Author

Execution Details

image

Modal

image

add support for displaying logs for runJob
Copy link
Contributor

@danielpeach danielpeach left a comment

Choose a reason for hiding this comment

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

@ethanfrogers
Copy link
Contributor Author

ethanfrogers commented Nov 21, 2017

@danielpeach i wanted to go that route, but since the logs are coming from pipeline context and not being read from the API, it wasn't possible.

@danielpeach
Copy link
Contributor

Oh you're absolutely right. Thanks.

@ethanfrogers
Copy link
Contributor Author

though, do you think it would be okay to extend the ConsoleOutputLink directive to take pre-defined logs instead of reading them from the API?

@ethanfrogers
Copy link
Contributor Author

i think i may just merge this and look at converging on the ConsoleOutputLInk directive in another PR. i have a feature i want to add to it anyway. is that cool, @danielpeach?

@danielpeach
Copy link
Contributor

Sounds good.

@ethanfrogers ethanfrogers merged commit 44c5fc9 into spinnaker:master Nov 21, 2017
@ethanfrogers ethanfrogers deleted the runjob-logs branch November 21, 2017 16:04
dkirillov added a commit to lookout/deck that referenced this pull request Dec 6, 2017
* feat(provider/kubernetes): runJob logs (spinnaker#4501)

add support for displaying logs for runJob

* feat(core): Add a pagerDuty read service, tag, and select field (spinnaker#4507)

* chore(core): Bump to 0.0.113 (spinnaker#4508)

* feat(artifacts): show artifacts next to manifest (spinnaker#4510)

* bug(core) - add missing div close that causes stage templates to not render. (spinnaker#4513)

* chore(core/pagerDuty): Add field to the pager duty service interface

* chore(core): Bump to 0.0.114

* fix(provider/amazon): Tags not copied when cloning ASG (spinnaker#4491)

* perf(core/executions): consider pipeline filters when fetching executions (spinnaker#4509)

* feat(core): console output jump to end (spinnaker#4511)

constrain modal height and add shortcut to end of the output by jumping
`pre` content to end.

* chore(core): fix lint warnings (spinnaker#4516)

* fix(core/executions): trigger filter collapse correctly (spinnaker#4515)

* chore(core): bump package to 0.0.115 (spinnaker#4517)

* chore(amazon): bump package to 0.0.52 (spinnaker#4518)

* fix(amazon): provide better error message when bake stage fails (spinnaker#4519)

* fix(core/pipeline): Pipeline graph overflow in firefox (spinnaker#4520)

* fix(*/loadBalancer): Fix server group show/hide control (spinnaker#4521)

* fix(*/loadBalancer): Fix a few undefined errors with load balancers (spinnaker#4524)

* fix(core/pipeline): Fix undefined edge case in ScriptExecutionDetails (spinnaker#4526)

* feat(kubernetes): export v2 provider (spinnaker#4523)

* chore(kubernetes): bump npm package version (spinnaker#4527)

* feat(webhooks): webhook trigger dialog (spinnaker#4522)

* chore(core): Fix lint (spinnaker#4529)

* fix(core/pipeline): De-duplicate state transitions for execution details (spinnaker#4531)

* fix(amazon/loadBalancer): Edit load balancers in the context of the application they were created in/with (spinnaker#4532)

* chore(core): bump to 0.0.116 (spinnaker#4533)

* chore(amazon): bump to 0.0.53 (spinnaker#4534)

* fix(core): reduce starting instance animations (spinnaker#4535)

*  refactor(core/pipeline): Convert singleExecutionDetails to react (spinnaker#4530)

* refactor(core/loadBalancer): Remove unused loadBalancers component

* refactor(core/pipeline): Convert singleExecutionDetails to react

* chore(core): bump package to 0.0.117 (spinnaker#4536)

* feat(provider/kubernetes): Add deploy manifest stage (spinnaker#4525)

* fix(provider/kubernetes): fixup deploy manifest stage (spinnaker#4538)

* fix(core): use inline element for instance groups (spinnaker#4540)

* chore(core): bump package to 0.0.118 (spinnaker#4541)

* fix(provider/kubernetes): v2 move backing data out of stage (spinnaker#4539)

* feat(pubsub): support constraints (spinnaker#4542)

* fix(core): prevent adding group stages, handle rendering gracefully (spinnaker#4543)

* chore(core): bump package to 0.0.119 (spinnaker#4544)

* feat(amazon): Support passing a capacity constraint during resize (spinnaker#4545)

`clouddriver` can enforce that capacity of the server group being
resized has not changed in the background.

This addresses situations where autoscaling has occurred in the
background and subsequently been "undone" by a concurrent resize operation.

* config(provider/openstack) - change default protocol for load balancer health check to HTTP. (spinnaker#4546)

* bug(provider/openstack) - filter networks for floating ip to the account selected. (spinnaker#4547)

* feat(gce): UI for zone selection. (spinnaker#4548)

* fix(artifacts): fix typo in artifact var (spinnaker#4549)

* refactor(core/pipeline): Cleanup time boundary sorting/comparison (spinnaker#4551)

* fix(provider/k8sv2): Move delete manifest form data into its own component (spinnaker#4550)

* chore(amazon): bump package to 0.0.54 (spinnaker#4553)

* feat(provider/kubernetes): deploy artifact from application (spinnaker#4552)

* fix(core/task): Do not show tasks link if no application associated with task (spinnaker#4556)

* refactor(core/entityTags): retrieve tags by application (spinnaker#4557)

* fix(amazon): omit spinnaker metadata tags when cloning server groups (spinnaker#4554)

* fix(core/pipelines): rerender view when first stage is removed (spinnaker#4558)

* chore(chore): bump package to 0.0.120 (spinnaker#4559)

* chore(amazon): bump package to 0.0.55 (spinnaker#4560)

* Removed some console.logs.
@odino
Copy link

odino commented Jan 9, 2018

When is this going to be available? :)

@ethanfrogers
Copy link
Contributor Author

@odino it will be available in the next minor release. i'm not sure if there's a date at this point though.

@odino
Copy link

odino commented Jan 9, 2018 via email

@gregdaybreak
Copy link

Did this ever go out in a release? I do not see the Logs section when I look in Run Job config. I am on version 1.6.1

screen shot 2018-04-20 at 4 18 36 pm

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.

4 participants