Skip to content

Commit

Permalink
test: Reduce length of log filenames
Browse files Browse the repository at this point in the history
With some filesystems (e.g., ecryptfs), the maximum filename length is
lower than 255, that of ext4.  On these filesystems, the tests fail with
the following error because log filenames are too long:

    K8sPolicyTest Basic Test Redirects traffic to proxy when no policy is applied with proxy-visibility annotation Tests HTTP proxy visibility without policy
    at /cilium/cilium/test/ginkgo-ext/scopes.go:430

    [Could not read monitor log
    Expected
        <*os.PathError | 0xc000e1fec0>: {
            Op: "open",
            Path: "test_results/117-/K8sPolicyTest_Basic_Test_Redirects_traffic_to_proxy_when_no_policy_is_applied_with_proxy-visibility_annotation_Tests_HTTP_proxy_visibility_without_policy/monitor-13f3cf9b-4f30-11ea-b222-60f262b6c493.log",
            Err: 0x24,
        }
    to be nil]

This commit updates the name of the test (from which the log filename is
derived) to be consistent with other tests' names.

Signed-off-by: Paul Chaignon <paul@cilium.io>
  • Loading branch information
pchaigno committed Feb 17, 2020
1 parent 6d47181 commit ff89671
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/k8sT/Policies.go
Expand Up @@ -792,7 +792,7 @@ var _ = Describe("K8sPolicyTest", func() {
})
})

Context("Redirects traffic to proxy when no policy is applied with proxy-visibility annotation", func() {
Context("Traffic redirections to proxy", func() {

var (
// track which app1 pod we care about, and its corresponding
Expand Down

0 comments on commit ff89671

Please sign in to comment.