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

Disallow colons #291

Merged
merged 32 commits into from
May 27, 2020
Merged

Disallow colons #291

merged 32 commits into from
May 27, 2020

Conversation

markphelps
Copy link
Collaborator

Fixes: #262

Screen Shot 2020-05-22 at 10 16 40 AM

* master:
  Import from STDIN (#287)
  Bump github.com/spf13/viper from 1.6.3 to 1.7.0 (#284)
  Bump github.com/Masterminds/squirrel from 1.3.0 to 1.4.0 (#285)
  Bump github.com/gofrs/uuid from 3.2.0+incompatible to 3.3.0+incompatible (#283)
  Bump github.com/lib/pq from 1.5.1 to 1.5.2
  Bump github.com/grpc-ecosystem/grpc-gateway from 1.14.4 to 1.14.5
  Bump github.com/golang/protobuf from 1.4.0 to 1.4.1
  Bump node-sass (#280)
  Bump github.com/golang/protobuf from 1.3.5 to 1.4.0 (#269)
  Bump github.com/golangci/golangci-lint from 1.24.0 to 1.26.0 (#275)
  Bump github.com/prometheus/client_golang from 1.5.1 to 1.6.0 (#277)
  Bump google.golang.org/grpc from 1.28.1 to 1.29.1 (#272)
  Bump github.com/Masterminds/squirrel from 1.2.0 to 1.3.0 (#276)
  Bump github.com/lib/pq from 1.3.0 to 1.5.1 (#279)
  Bump github.com/sirupsen/logrus from 1.5.0 to 1.6.0 (#273)
@codecov-commenter
Copy link

codecov-commenter commented May 22, 2020

Codecov Report

Merging #291 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #291   +/-   ##
=======================================
  Coverage   83.44%   83.44%           
=======================================
  Files          19       19           
  Lines        1679     1679           
=======================================
  Hits         1401     1401           
  Misses        190      190           
  Partials       88       88           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8ea824...a8ea824. Read the comment docs.

@markphelps
Copy link
Collaborator Author

@jperl 👋🏻 I was wondering if you could help me debug why my QAWolf Integration Test is failing here: https://github.com/markphelps/flipt/runs/709580872?check_suite_focus=true#step:10:115

I am trying to merge the qawolf.yml workflow that you created with my existing integration-test.yml workflow, so that QAWolf will always run against the latest (changed) code, instead of having to use the Docker image that I publish on release.

It seems that QAWolf/Playwright is getting a 404 Not Found when trying to navigate to http://0.0.0.0:8080 within the container.. however I can't think why that would be the case, as I wait for the host to be available here: https://github.com/markphelps/flipt/pull/291/files#diff-082650297ecabbdf711809e2fcc86731R21

Attached is the zip containing the video from ffmpeg (which is awesome btw. super helpful for debugging):

qawolf-1.zip

Any help would be greatly appreciated!

@jperl
Copy link
Contributor

jperl commented May 26, 2020

@markphelps Can you run curl http://0.0.0.0:8080 after the wait-for-it so we can see what the output is?

…into disallow-colons

* 'disallow-colons' of https://github.com/markphelps/flipt:
  Bump github.com/grpc-ecosystem/grpc-gateway from 1.14.5 to 1.14.6
  Bump gopkg.in/yaml.v2 from 2.2.8 to 2.3.0 (#290)
  Bump github.com/golang/protobuf from 1.4.1 to 1.4.2
@markphelps
Copy link
Collaborator Author

@jperl I figured it out, I wasn't 'building the ui' in the Integration Test workflow.. so it had nothing to serve, hence the 404.

Thanks!

@jperl
Copy link
Contributor

jperl commented May 27, 2020

Ok great. Glad you figured it out 🙏. Thanks for the update.

@markphelps markphelps merged commit e3bb45c into master May 27, 2020
@markphelps markphelps deleted the disallow-colons branch May 27, 2020 13:43
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.

Flags with special characters
3 participants