Revision 230f0b9b7cb4885864d457c82e312489c0a0d3d7 authored by Paul Chaignon on 21 May 2021, 21:10:24 UTC, committed by Nathan Sweet on 04 June 2021, 16:46:12 UTC
[ upstream commit 802d25447fa1d325d3713220f8ad08e7938f6dd4 ]

Commit 50df544 added a new job in the end-to-end workflows to checkout
the tested code and run paths-filter on it. That first job fails when
the workflows are scheduled (vs. triggered by PR comment) because we try
to grab the pull request URL, which doesn't exist:

    curl ${{ github.event.issue.pull_request.url }} > pr.json

results in error:

    Run curl  > pr.json
    curl: try 'curl --help' or 'curl --manual' for more information

We can avoid this by skipping steps in the first job when triggered by
schedule. We can't skip the entire first job because the second job has
a dependency on the first and would fail if we skipped the first.

Fixes: 50df544 (".github: Skip unnecessary ci-xxx tests")
Signed-off-by: Paul Chaignon <paul@cilium.io>
Signed-off-by: Quentin Monnet <quentin@isovalent.com>
1 parent e942366
Raw File
Makefile
# Copyright 2017-2020 Authors of Cilium
# SPDX-License-Identifier: Apache-2.0

include ../Makefile.defs

TARGETS := cilium-operator cilium-operator-generic cilium-operator-aws cilium-operator-azure cilium-operator-alibabacloud

.PHONY: all $(TARGETS) clean install

all: $(TARGETS)

cilium-operator: GO_TAGS_FLAGS+=ipam_provider_aws,ipam_provider_azure,ipam_provider_operator,ipam_provider_alibabacloud
cilium-operator-generic: GO_TAGS_FLAGS+=ipam_provider_operator
cilium-operator-aws: GO_TAGS_FLAGS+=ipam_provider_aws
cilium-operator-azure: GO_TAGS_FLAGS+=ipam_provider_azure
cilium-operator-alibabacloud: GO_TAGS_FLAGS+=ipam_provider_alibabacloud

$(TARGETS):
	@$(ECHO_GO)
	$(QUIET)$(GO_BUILD) -o $(@)

$(TARGET):
	@$(ECHO_GO)
	$(QUIET)$(GO_BUILD) -o $@

clean:
	@$(ECHO_CLEAN)
	$(QUIET)rm -f $(TARGETS)
	$(GO) clean $(GOCLEAN)

install:
	$(QUIET)$(INSTALL) -m 0755 -d $(DESTDIR)$(BINDIR)
	$(foreach target,$(TARGETS), $(QUIET)$(INSTALL) -m 0755 $(target) $(DESTDIR)$(BINDIR);)
back to top