Skip to content

Buildpack logs to STDERR when doing normal operations #27

Closed

Description

@pivotal-feedback

It's kinda scary to watch the log output as an app is deploying when there's a big red block of text. Seems like it should all go to STDOUT:

2014-12-19T16:50:57.48-0800 [STG]     OUT -----> Downloaded app buildpack cache (66M)
2014-12-19T16:51:01.51-0800 [STG]     ERR Cloning into '/tmp/buildpacks/ruby-buildpack'...
2014-12-19T16:51:02.84-0800 [STG]     ERR Note: checking out '48a89f7ed6814cfce869bbaf9acb945baa10bd2f'.
2014-12-19T16:51:02.84-0800 [STG]     ERR You are in 'detached HEAD' state. You can look around, make experimental
2014-12-19T16:51:02.84-0800 [STG]     ERR changes and commit them, and you can discard any commits you make in this
2014-12-19T16:51:02.84-0800 [STG]     ERR state without impacting any branches by performing another checkout.
2014-12-19T16:51:02.84-0800 [STG]     ERR If you want to create a new branch to retain commits you create, you may
2014-12-19T16:51:02.84-0800 [STG]     ERR do so (now or later) by using -b with the checkout command again. Example:
2014-12-19T16:51:02.84-0800 [STG]     ERR   git checkout -b new_branch_name
2014-12-19T16:51:03.05-0800 [STG]     OUT Submodule 'buildpack-packager' (https://github.com/cf-buildpacks/buildpack-packager.git) registered for path 'buildpack-packager'
2014-12-19T16:51:03.10-0800 [STG]     ERR Cloning into 'buildpack-packager'...
2014-12-19T16:51:03.38-0800 [STG]     OUT Submodule path 'buildpack-packager': checked out 'fec9c4598a6496fb9fa02c288076dc120754b0b5'

Activity

  1. cf-gitbot commented on Dec 20, 2014

    @cf-gitbot

    We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/84963100.

  2. jchesterpivotal commented on Dec 22, 2014

    @jchesterpivotal
    Contributor

    This behaviour comes from git, which loves to spray status updates into STDERR. See for example this discussion.

    I'm personally wary of tinkering with the git defaults, but I also hate spurious errors. I don't consider this bug, we might reschedule it as a story.

  3. jchesterpivotal commented on Dec 24, 2014

    @jchesterpivotal
    Contributor

    We've scheduled a story to change Git's behaviour to only show up on STDOUT. I'll close this issue once we're done.

  4. emalm commented on Dec 24, 2014

    @emalm

    Is the DEA's buildpack runner the code actually performing the git clone here? https://github.com/cloudfoundry/dea_ng/blob/master/buildpacks/lib/git.rb

    What's the cf push command used to push this app? Does it specify a buildpack?

  5. pivotal-feedback commented on Dec 29, 2014

    @pivotal-feedback
    Author

    The buildpack is not explicitly specified as part of the push command, but
    it is implicitly specified (to ruby-buildpack v1.1.3 in our case) due to
    the way CF caches buildpacks.

    ~Will
    Engineering Manager
    Pivotal Labs

    On Wed, Dec 24, 2014 at 10:13 AM, Eric Malm notifications@github.com
    wrote:

    Is the DEA's buildpack runner the code actually performing the git clone
    here?
    https://github.com/cloudfoundry/dea_ng/blob/master/buildpacks/lib/git.rb

    What's the cf push command used to push this app? Does it specify a
    buildpack?

    —
    Reply to this email directly or view it on GitHub
    #27 (comment)
    .

  6. squeedee commented on Jan 27, 2015

    @squeedee
    Contributor
  7. Dannyzen commented on Dec 30, 2015

    @Dannyzen
    Contributor

    This story was completed and this is good to close

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions