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 does not work locally #161

Closed
edwinpjacques opened this issue Oct 27, 2020 · 2 comments
Closed

CI does not work locally #161

edwinpjacques opened this issue Oct 27, 2020 · 2 comments

Comments

@edwinpjacques
Copy link
Contributor

edwinpjacques commented Oct 27, 2020

Expected behaviour

When I run "make lint", "make test" or "make ci" on a Linux host, I expect everything to work.

Actual behaviour

make lint dies on cpplint

libtool: link: ( cd "samples/.libs" && rm -f "libsamples.la" && ln -s "../libsamples.la" "libsamples.la" )
[root@ejacques-mbp-centos restclient-cpp]# make ci
cpplint --filter=-legal/copyright include/restclient-cpp/*.h source/*.cc
.
.
.
include/restclient-cpp/restclient.h:9:  #ifndef header guard has wrong style, please use: TF_DEV_ENV_CONTRAIL_THIRD_PARTY_RESTCLIENT_CPP_INCLUDE_RESTCLIENT_CPP_RESTCLIENT_H_  [build/header_guard] [5]
include/restclient-cpp/restclient.h:68:  #endif line should be "#endif  // TF_DEV_ENV_CONTRAIL_THIRD_PARTY_RESTCLIENT_CPP_INCLUDE_RESTCLIENT_CPP_RESTCLIENT_H_"  [build/header_guard] [5]
Done processing include/restclient-cpp/restclient.h
Done processing source/connection.cc
Done processing source/helpers.cc
Done processing source/restclient.cc
Total errors found: 6
make: *** [lint] Error 1

make test has three test failures, and one ignored test... it's unclear to the tester why.

[  FAILED  ] 3 tests, listed below:
[  FAILED  ] ConnectionTest.TestFollowRedirect
[  FAILED  ] ConnectionTest.TestFollowRedirectLimited
[  FAILED  ] ConnectionTest.TestGetInfoFromRedirect

 3 FAILED TESTS
  YOU HAVE 1 DISABLED TEST

make: *** [test] Error 1

I also get the same results above if I run the command make docker-services test.

make ci fails because of above errors.

Environment and debugging details

  • gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-39)
  • CentOS Linux 7
  • 5.2
  • git install from master
  • libcurl version 7.29.0-57.el7_8.1 (vendor-provided)
  • full error output above
@edwinpjacques edwinpjacques changed the title CI is broken CI does not work locally Oct 27, 2020
@edwinpjacques
Copy link
Contributor Author

Addressed this issue in PR #160

@edwinpjacques
Copy link
Contributor Author

fix merged

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

No branches or pull requests

1 participant