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

fixconfig fail to build? #7652

Closed
krzyzacy opened this issue Apr 11, 2018 · 6 comments
Closed

fixconfig fail to build? #7652

krzyzacy opened this issue Apr 11, 2018 · 6 comments

Comments

@krzyzacy
Copy link
Member

senlu@senlu:~/work/src/k8s.io/test-infra$ ./hack/update-config.sh 
INFO: Analysed target //maintenance/fixconfig:fixconfig (0 packages loaded).
INFO: Found 1 target...
ERROR: Process exited with status 127: Process exited with status 127
/bin/sh: print-workspace-status.sh: command not found
Target //maintenance/fixconfig:fixconfig failed to build
Use --verbose_failures to see the command lines of failed build steps.

/area bazel
cc @BenTheElder

@krzyzacy
Copy link
Member Author

cc @ixdy

@ixdy
Copy link
Member

ixdy commented Apr 11, 2018

we can work around this by using --workspace_status_command="bash ./print-workspace-status.sh" instead of just --workspace_status_command=./print-workspace-status.sh.

I'm not sure if this was an intentional regression; I'm going to open an issue against bazel.

@BenTheElder
Copy link
Member

our current usage seems to match the docs :|
https://docs.bazel.build/versions/master/user-manual.html#workspace_status

Excerpt:


Example program on Linux using Git:

#!/bin/bash
echo "CURRENT_TIME $(date +%s)"
echo "RANDOM_HASH $(cat /dev/urandom | head -c16 | md5sum 2>/dev/null | cut -f1 -d' ')"
echo "STABLE_GIT_COMMIT $(git rev-parse HEAD)"
echo "STABLE_USER_NAME $USER"

Pass this program's path with --workspace_status_command, and the stable status file will include the
STABLE lines and the volatile status file will include the rest of the lines.

@ixdy
Copy link
Member

ixdy commented Apr 11, 2018

bazelbuild/bazel#5002

@ixdy
Copy link
Member

ixdy commented Apr 11, 2018

but kubernetes/kubernetes isn't broken? this is... bizarre.

@ixdy
Copy link
Member

ixdy commented Apr 11, 2018

oh, it gets better. if we move print-workspace-status.sh into hack/ and then update --workspace_status_command=./hack/print-workspace-status.sh then it works again.

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

4 participants