{"push_id":4232537720,"size":2,"distinct_size":0,"ref":"refs/heads/get-rid-of-checklicence-image","head":"3e4348bc6e5cfd4a258eb3c7e611a4eeaff4c87b","before":"9c794ab6033e56acb52750a6e5f299fb54cc2f26","commits":[{"sha":"39f13ea8b0b3ee59739ec50e23d423b93d25eab6","author":{"name":"Jarek Potiuk","email":"f41fb3c6021305a2b88483c10bc35385d65a7c7a@polidea.com"},"message":"[AIRFLOW-5830] Get rid of slim image\n\nThe slim image gave only very small gain on executing the tests in CI. The\nimage was significantly smaller, but then for local development and testing\nyou needed both full CI and SLIM-CI image.\n\nThis made the scripts and docker image needlessly complex - especially\nin the wake of coming Production image it turned to be premature\noptimisation really. While it sped-up (slightly - by 10-20 seconds) some\nstatic check jobs in Travis, it increased time needed by developers\nto have a working environment and to keep it updated every time it was\nneeded (by minutes)\n\nAlso having two separately managed images made it rather complex to join\nsome of the Travis CI jobs (there is a follow-up change with getting rid\nof Checklicence image).\n\nWith this change both static checks and tests are executed using single\nimage. That also opens doors for further simplification of the scripts\nand easier implementation of production image.","distinct":false,"url":"https://api.github.com/repos/PolideaInternal/airflow/commits/39f13ea8b0b3ee59739ec50e23d423b93d25eab6"},{"sha":"3e4348bc6e5cfd4a258eb3c7e611a4eeaff4c87b","author":{"name":"Jarek Potiuk","email":"f41fb3c6021305a2b88483c10bc35385d65a7c7a@polidea.com"},"message":"[AIRFLOW-5829] Get rid of the checklicence image\n\nThis change is a further step of simplifying the set of scripts\nused by CI. The separate checklicence image was implemented as an\noptimisation of the licence check time. The image to download was\nsmall and could be downloaded slightly faster in CI. However that\nmade all the management script more complex and lead to having\nseparate jobs for check licence and static checks. That lead to\nactually longer time of Travis jobs - because new machine had to\nbe spun-off for checklicence check only.\n\nWith this change, the CI image is the only one left and it is slightly\nbigger (with RAT tool added) but the same image is used for all the\ntests - unit tests, static checks and checklicence checks.\n\nThis also makes it easier to manage the images and decreases update\noverhead on the developers using Breeze.","distinct":false,"url":"https://api.github.com/repos/PolideaInternal/airflow/commits/3e4348bc6e5cfd4a258eb3c7e611a4eeaff4c87b"}]}
{"actor":{"display_login":"potiuk"}}