varnishcache-friends / libvmod-geoip2

Varnish VMOD to query MaxMind GeoIP2 DB files
BSD 2-Clause "Simplified" License
39 stars 17 forks source link

Test b00002.vtc fails with Varnish 6.0.4 #34

Closed jnk0 closed 5 years ago

jnk0 commented 5 years ago

Trying to build libvmod-geoip2 with Varnish 6.0.4. But test b00002.vtc fails.

Relevant output from test-suite.log:

***  v1    1.1 debug|Debug: Version: varnish-6.0.4 revision 14d4eee6f0afc3020a044341235f54f3bd1449f1
***  v1    1.1 debug|Debug: Platform: Linux,3.10.0-957.27.2.el7.x86_64,x86_64,-junix,-sdefault,-sdefault,-hcritbit
***  v1    1.1 debug|200 327     
***  v1    1.1 debug|-----------------------------
***  v1    1.1 debug|Varnish Cache CLI 1.0
***  v1    1.1 debug|-----------------------------
***  v1    1.1 debug|Linux,3.10.0-957.27.2.el7.x86_64,x86_64,-junix,-sdefault,-sdefault,-hcritbit
***  v1    1.1 debug|varnish-6.0.4 revision 14d4eee6f0afc3020a044341235f54f3bd1449f1
***  v1    1.1 debug|
***  v1    1.1 debug|Type 'help' for command list.
***  v1    1.1 debug|Type 'quit' to close CLI session.
***  v1    1.1 debug|Type 'start' to launch worker process.
***  v1    1.1 debug|

...

***  v1    2.9 debug|VCL "vcl1" Failed initialization
***  v1    2.9 debug|Message:
***  v1    2.9 debug|\tObject vo_db2 not initialized
***  v1    2.9 debug|
***  v1    2.9 debug|Debug: Stopping Child
**** v1    3.0 vsl|          0 CLI             - Rd vcl.load "vcl1" vcl_vcl1.1569759457.139654/vgc.so 1auto
**** v1    3.0 vsl|          0 Debug           - geoip2.geoip2: Using maxminddb 1.2.0
**** v1    3.0 vsl|          0 Debug           - geoip2.geoip2: Using maxminddb 1.2.0
**** v1    3.0 vsl|          0 Error           - geoip2.geoip2: Error opening the specified MaxMind DB file
**** v1    3.0 vsl|          0 CLI             - Wr 300 73 VCL "vcl1" Failed initialization
Message:
        Object vo_db2 not initialized

**** v1    3.0 vsl|          0 CLI             - EOF on CLI connection, worker stops
jnk0 commented 5 years ago

Build with Varnish 6.0.3 works fine, test b00002.vtc is passed.

jnk0 commented 5 years ago

Adding full test-suite.log

fgsch commented 5 years ago

Thanks for the report. Which branch are you using?

jnk0 commented 5 years ago

I am using branch oldstable.

fgsch commented 5 years ago

Thanks. I will take a look and update this ticket.

fgsch commented 5 years ago

Can you try using devel?

Now that they have more or less converged I will unify the development in a single branch (probably master) soon.

jnk0 commented 5 years ago

Using branch devel the build process passes all tests.

fgsch commented 5 years ago

Thanks for confirming it.

I've move the support for 6.0, 6.2 and 6.3 to master, and tagged v1.2.1.

Please use that going forward.