Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 14 12:30
    justb4 labeled #46
  • Oct 14 12:30
    justb4 commented #46
  • Oct 13 12:41
    PB-GNM commented #46
  • Oct 13 07:43

    github-actions[bot] on gh-pages

    Update website via GH Action (compare)

  • Oct 13 07:42

    justb4 on main

    correct database name to gis lo… correct github branch name to m… PR #47 from thijsbrentjens/main… (compare)

  • Oct 13 07:42
    justb4 closed #47
  • Oct 13 07:41
    justb4 labeled #47
  • Oct 13 07:41
    justb4 labeled #47
  • Oct 13 07:41
    justb4 assigned #47
  • Oct 13 07:40
    justb4 review_request_removed #47
  • Oct 13 07:40
    justb4 review_requested #47
  • Oct 13 07:40
    justb4 review_requested #47
  • Oct 13 07:28
    thijsbrentjens opened #47
  • Sep 21 15:26
    PB-GNM opened #46
  • Sep 21 11:10

    github-actions[bot] on gh-pages

    Update website via GH Action (compare)

  • Sep 21 11:09

    justb4 on main

    update docs with OpenGeodag 202… (compare)

  • Sep 21 10:50
    justb4 closed #45
  • Sep 21 10:50

    justb4 on main

    #45 force Python3 for Ansible … (compare)

  • Sep 21 10:46
    justb4 labeled #45
  • Sep 21 10:46
    justb4 assigned #45
paul van genuchten
@pvgenuchten
image.png
Just, heb jij een idee waarom het pygeoapi image zo groot is (in vergelijking met bv het pycsw image)
paul van genuchten
@pvgenuchten
seems an extended version of linux, the /usr/lib/locale folder has 255mb
Just van den Broecke
@justb4
ja ben ik ook niet blij mee, heeft met ook met base image te maken, was blijkbaar nodig voor laatste GDAL, en idd mogelijk alle locales die je niet nodig hebt.
alpine base image gebruik ik normaal maar is te lastig hier.
Plus: http://schemas.opengis.net/SCHEMAS_OPENGIS_NET.zip zit er ook in waar ik niet blij mee was.
Denk dat we zeker kunnen 'prunen'. maar was ooit 1.3GB dus al beter, maar moet zeker kleiner.
maar moeten we bij pygeoapi bespreken.
paul van genuchten
@pvgenuchten
+1
This message was deleted
paul van genuchten
@pvgenuchten
vanmorgen standup gedaan met pdok, pdok werkt aan een opensource go implementatie https://github.com/PDOK/goaf, is dat iets om ook aan het testbed toe te voegen, pieter stelde ook voor om deze implementatie op te nemen in https://apitestdocs.geonovum.nl/cases/INSPIRE/
Just van den Broecke
@justb4
ok, begrijp nog steeds niet waarom ze niet gewoon pygeoapi nemen/verbeteren... zeker als je 100% compliant met ook nog INSPIRE-eisen wilt zijn...wij weten wat daar bij komt kijken...de ontwikkel-resources zijn al te beperkt voor nog meer versnippering...GOAF kwam zelfs niet door de simpelste GeoHealthCheck test...maar goed...laten we GOAF in stack opnemen. Mijn uren zijn echter beperkt (28 left).
Thijs Brentjens
@thijsbrentjens
GOAF opnemen wat mij betreft als er tijd voor is :). Eerst de rest goed afronden. Daarna is het wel aardig/interessant om te doen. Ook om te laten zien wat verschillen zijn met andere implementaties. Kan ook trouwens dat we PDOK vragen dit op te nemen, samen met ons. Dus dat PDOK een Docker container voorbereidt. En misschien probeert toe te voegen aan de omgeving
paul van genuchten
@pvgenuchten
Goaf is als docker beschikbaar, zelfs inclusief geconfigureerde data
paul van genuchten
@pvgenuchten
Ik kijk of ik goaf in n uurtje werkend krijg, daarna delegeer ik graag naar pdok
Thijs Brentjens
@thijsbrentjens
Okay. Leuk als het lukt, maar als het niet lukt, liever ook weer weghalen (zoals bij qgis dat net niet werkt), anders vetroebelt dat net wat veel
Just van den Broecke
@justb4
qgis werkt nu, ook in Sandbox, maar met shortcut: chmod 777 data...
gisteren in workshop waren er bevindingen en verbeteringen om aantal stappen in opzet nieuwe service te verkleinen.
@thijsbrentjens heeft hopelijk een lijstje. Wat mij vooral bijbleef was aantal wijzigingen voor bijv pygeoapi in docker-compose.yml die met vars enenv.sh bestand kunnen. Moet ik daarnaar kijken? Andere was vooral nog wat HOWTO docu detail nodig.
paul van genuchten
@pvgenuchten
Ben benieuwd naar je lijstje Thijs, heb je ook de bridge-geoserver use case doorgenomen?
Ik begreep dat ine en Pieter de sessie als technisch ervaren hadden, mogelijk is de bridge-geoserver case iets voor hen
Just van den Broecke
@justb4
@pvgenuchten het ging meer om de basic workflow rond wijzigingen maken en daarvan resultaat zien. Use cases met pygeoapi service: 1) tekst metadata wijzigen 2) nieuwe pygeoapi service "bgt", (BGT Picknicktafels GPKG) maken en deployen. Dat gaf al genoeg "gotcha's":
  • nieuwe service: steeds 3 zaken wijzigen: service directory, Ansible deploy.yml en GH Action /.github/workflows/deploy.service.yml
  • via PR kan geen nieuwe Workflow gemaakt/gemerged
  • PRs iha bleken erg omslachtig dus direct push naar main in Sandbox
  • bij dupliceren service directory ging executable modus van .sh files verloren
  • toch erg veel identieke wijzigingen m.n. string pygeoapi naar bgt in docker-compose.yml
  • de Traefik routing labels in docker-compose.yml moeten ook nieuwe id's krijgen bijv ...routers.pygeoapi_https.rule... etc, anders conflict
De bullet 5+6 is wel te verhelpen met shell/Docker variabelen. De overige meer aanscherpen HOWTOs.
Thijs Brentjens
@thijsbrentjens
belangrijkste zijn die env vars @justb4 , kom er morgen op teru
Just van den Broecke
@justb4
PoC gedaan: werkt, zie:
Geonovum/ogc-api-testbed#32
Just van den Broecke
@justb4
en https://github.com/Geonovum/ogc-api-testbed/tree/main/services/pygeoapi als eerste. Als zo ok, is het werk meer om dit 9x te herhalen op Stable en Sandbox en te documenteren. Maar is nu 1 regel wijziging in env.sh.
Thijs Brentjens
@thijsbrentjens
Het ziet er netjes uit Just. Ik zal het proberen nog eens te testen voor de bgt demo bijvoorbeeld :).
paul van genuchten
@pvgenuchten
ik heb zojuist (een wat oudere versie van) goaf gedeployed op https://apitestbed.geonovum.nl/goaf/collections?f=html
Thijs Brentjens
@thijsbrentjens
Mooi @pvgenuchten . Over de laatste dingen: hopelijk kunnen jullie verder. @justb4 komen de aanpassingen uit de prod omgeving voor makkelijker inrichten via env-vars ook naar de sandbox?
Andere punten na onze sessie van woensdag zijn vooral documentatie gereateerd en staan op github als issues.
Qua planning: ik ben maandag beperkt beschikbaar, dinsdag niet (dinsdagen in principe altijd vrij, maar nu ook echt de dag weg). Woensdag wel, zullen we dan een afspraak maken ter afronding, om dingen door te nemen?
paul van genuchten
@pvgenuchten
ik geef woensdag ochtend een training, ben wel in de middag beschikbaar
ik loop de issues na, en assign ze aan mezelf toe, als ik ermee aan de slag ga
paul van genuchten
@pvgenuchten
@justb4 in relatie tot Geonovum/ogc-api-testbed#23, MIT (en/of CC-by voor docs) doen?
paul van genuchten
@pvgenuchten
inspire implementatie in pygeoapi is verbeterd, links naar metadata, bulk download en feature concept thesaurus opgenomen, https://apitestbed.geonovum.nl/pygeoapi/collections/ProtectedSites
En er worden brt tegels van pdok als achtergrondje gebruikt (leek me aardig aanvulling)
Thijs Brentjens
@thijsbrentjens
Mooi werk. Ik probeer zo te reageren op een paar issues. Vanmiddag heb ik trouwens ook gelegenheid om dingen door te praten, als jullie daar nog behoefte aan hebben. Vanaf half 3 / 3 uur denk ik
paul van genuchten
@pvgenuchten
helaas ben ik vanmiddag niet in de gelegenheid, is woensdag middag een optie?
(of morgen)
Thijs Brentjens
@thijsbrentjens
s middags heb ik een vector tiling sessie. Om 13 uur kan nog wel
morgen kan ik niet (werk niet op dinsdagen en morgen helemaal lastig omdat ik een dag weg ben met de kinderen, studiedag van school)
Just van den Broecke
@justb4

ik was even vrij, vanaf nu weer aan de slag. zag veel, vooral van @pvgenuchten langskomen, mooi!
ik kan vanmiddag (met @thijsbrentjens ) en ook woensdag (alledrie) voor overleg. wo in Bennekom @pvgenuchten ?

Denk dat voor nu "het env vars principe" voor alle docker-compose.yml files in Stable en Sandbox hoogste prio heeft. Dan begin ik daar nu aan. Is even werkje. Woensdag 30 juni toch deadline?

Thijs Brentjens
@thijsbrentjens
woe inderdaad deadline. Prio 1 is die env vars aanpakken ja. En dan misschien een rondje docs nalopen, of er nog (kleine) aanpassingen nodig zijn. Dan is deze fase afgerond denk ik. Als die env vars staan, doe ik nog een korte test, kijken of het echt zo mooi werkt :). Zullen we anders woensdagmiddag even overleggen?
Just van den Broecke
@justb4
Ja is goed, wo middag. Moet GOAF ook op Sandbox?
Thijs Brentjens
@thijsbrentjens
Als het makkelijk kan graag, anders hoeft het (nog) niet
paul van genuchten
@pvgenuchten

ik kan vanmiddag (met @thijsbrentjens ) en ook woensdag (alledrie) voor overleg. wo in Bennekom @pvgenuchten ?

Ik geef woensdag ochtend een cursus, woensdag middag van harte welkom

merge main > sandbox hoort eenvoudig te gaan
Just van den Broecke
@justb4
ok, merge uit main naar sandbox tricky, weet ook niet of dat kan bij GH Template.
eerste weer permissie-probleem GeoServer bij redeploy oplossen...
Just van den Broecke
@justb4
ok, de "env var strategie" uit #32 is doorgevoerd in Stable en Sandbox.
De documentatie hiervoor staat in: https://apitestdocs.geonovum.nl/howto/howto_deploy/
Ook de GHC doc is updated: https://apitestdocs.geonovum.nl/howto/howto_ghc/
Just van den Broecke
@justb4
En de docs gebruikt Material for mkdocs Theme: https://apitestdocs.geonovum.nl/ . Geeft ook beter onderscheid met de landings-pagina's. Thema toevoegen is erg eenvoudig overigens.
Just van den Broecke
@justb4
Op de Sandbox faalden alle Actions. Dit is niet fataal: was in post-deploy cleanup, dus uitrollen ging wel goed. Had te maken met pygeoapi: de deploy.sh probeert nieuw image op te halen. Andere instanties draaien nog op oude image. De git/services/clean.sh probeert "dangling" images weg te gooien maar faalt. Die error propageert naar de Action. Herstart van pygeoapi was tijdelijke oplossing.
Echte Oplossing: clean.sh moet fouten negeren. Zou opgelost moeten zijn via #36.
Dus GOAF draait ook op Sandbox. Ook voor algemene system maintenance (OS packages updaten, service herstarten, zelfs reboot via Ansible) HOWTO:
https://apitestdocs.geonovum.nl/howto/howto_system/ (zie #33).
Thijs Brentjens
@thijsbrentjens
Lekker bommetje nog zo einde dag om weg te halen . Mooi dat dat gelukt is