Skip to content

make: fix LOCKDEBUG env variable reference for docker-plugin-image#12318

Merged
pchaigno merged 1 commit intomasterfrom
pr/rolinh/docker-plugin-fix-lockdebug
Jun 29, 2020
Merged

make: fix LOCKDEBUG env variable reference for docker-plugin-image#12318
pchaigno merged 1 commit intomasterfrom
pr/rolinh/docker-plugin-fix-lockdebug

Conversation

@rolinh
Copy link
Copy Markdown
Member

@rolinh rolinh commented Jun 29, 2020

This commit fixes a typo that prevents LOCKDEBUG from working for the docker-plugin-image make target.

Note for backporters: in v1.7, the change has to be applied on Makefile. Look for LOCKDEUBG and replace it.

@rolinh rolinh added release-note/bug This PR fixes an issue in a previous release of Cilium. needs-backport/1.7 area/build Anything to do with the build, more general than area/CI labels Jun 29, 2020
@rolinh rolinh requested a review from a team as a code owner June 29, 2020 12:28
@rolinh
Copy link
Copy Markdown
Member Author

rolinh commented Jun 29, 2020

make: fix LOCKEDBUG

I managed to come up with yet another typo in the commit description... 🤦 Oh well 😛

This commit fixes a typo that prevents LOCKDEBUG from working for the
`docker-plugin-image` make target.

Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
@rolinh rolinh force-pushed the pr/rolinh/docker-plugin-fix-lockdebug branch from e2731cb to 9ff4a38 Compare June 29, 2020 12:58
@rolinh rolinh changed the title make: fix LOCKEDBUG env variable reference for docker-plugin-image make: fix LOCKDEBUG env variable reference for docker-plugin-image Jun 29, 2020
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 29, 2020

Coverage Status

Coverage decreased (-0.02%) to 36.915% when pulling 9ff4a38 on pr/rolinh/docker-plugin-fix-lockdebug into 48f8e79 on master.

@pchaigno
Copy link
Copy Markdown
Member

@rolinh Is this covered by any e2e test?

@rolinh
Copy link
Copy Markdown
Member Author

rolinh commented Jun 29, 2020

@pchaigno I don't think so which is why I didn't trigger the whole test suite.

@pchaigno
Copy link
Copy Markdown
Member

Let's merge then!

@pchaigno pchaigno merged commit 8cf1e20 into master Jun 29, 2020
@pchaigno pchaigno deleted the pr/rolinh/docker-plugin-fix-lockdebug branch June 29, 2020 15:44
@christarazi
Copy link
Copy Markdown
Member

christarazi commented Jun 30, 2020

Just wanted to mention that I appreciate your backporting note. It saved me time & effort while doing 1.7 backports. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build Anything to do with the build, more general than area/CI release-note/bug This PR fixes an issue in a previous release of Cilium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants