Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rendering SLS missing pillar key flakies #2444

Closed
TeddyAndrieux opened this issue Apr 22, 2020 · 0 comments
Closed

Rendering SLS missing pillar key flakies #2444

TeddyAndrieux opened this issue Apr 22, 2020 · 0 comments
Assignees
Labels
complexity:easy Something that requires less than a day to fix kind:bug Something isn't working topic:flakiness Some test are flaky and cause CI to do transient failing topic:lifecycle Issues related to upgrade or downgrade of MetalK8s

Comments

@TeddyAndrieux
Copy link
Collaborator

TeddyAndrieux commented Apr 22, 2020

Component:

'salt'

What happened:

Time to time we got flaky error during upgrade, downgrade.

This is due to Endpoint not having any Ip, ports, ...

{
    "apiVersion": "v1",
    "kind": "Endpoints",
    "metadata": {
        "creationTimestamp": "2020-04-17T09:25:32Z",
        "labels": {
            "app": "repositories",
            "app.kubernetes.io/managed-by": "salt",
            "app.kubernetes.io/name": "repositories",
            "app.kubernetes.io/part-of": "metalk8s",
            "heritage": "metalk8s",
            "metalk8s.scality.com/version": "2.4.2"
        },
        "name": "repositories",
        "namespace": "kube-system",
        "resourceVersion": "898628",
        "selfLink": "/api/v1/namespaces/kube-system/endpoints/repositories",
        "uid": "c8181dad-e58a-4772-9e40-5e86a23ab165"
    }
}

What was expected:

No more flaky

Steps to reproduce

Flaky

Resolution proposal (optional):

Some try for a fix was already done 8feca89 but refresh_pillar is not synchronous instead of using refresh_pillar the check_pillar_keys function should import salt and use directly salt.pillar.get_pillar function.


Note: A fresh pillar (with all ext_pillar) is gathered by salt-minion each and every time a salt state is executed

@TeddyAndrieux TeddyAndrieux added kind:bug Something isn't working topic:flakiness Some test are flaky and cause CI to do transient failing topic:lifecycle Issues related to upgrade or downgrade of MetalK8s complexity:easy Something that requires less than a day to fix labels Apr 22, 2020
@TeddyAndrieux TeddyAndrieux self-assigned this Apr 22, 2020
TeddyAndrieux added a commit that referenced this issue Apr 22, 2020
In `check_pillar_keys` gather pillar information using internal salt
function instead of using `saltutil.refresh_pillar` as `refresh_pillar`
do not allow synchronous refresh in salt 2018.3

Fixes: #2444
TeddyAndrieux added a commit that referenced this issue Apr 22, 2020
…rate"

This reverts commit f70905c.

Refresh_pillar is not needed as a pillar is gathered by the salt-minion
each time a state is run

Sees: #2444
bert-e added a commit that referenced this issue Apr 22, 2020
…into tmp/octopus/w/2.2/bugfix/GH-2444-gather-pillar-instead-of-refresh
TeddyAndrieux added a commit that referenced this issue Apr 22, 2020
In `check_pillar_keys` gather pillar information using internal salt
function instead of using `saltutil.refresh_pillar` as `refresh_pillar`
do not allow synchronous refresh in salt 2018.3

Fixes: #2444
TeddyAndrieux added a commit that referenced this issue Apr 22, 2020
…rate"

This reverts commit f70905c.

Refresh_pillar is not needed as a pillar is gathered by the salt-minion
each time a state is run

Sees: #2444
bert-e added a commit that referenced this issue Apr 22, 2020
…into tmp/octopus/w/2.2/bugfix/GH-2444-gather-pillar-instead-of-refresh
TeddyAndrieux added a commit that referenced this issue Apr 22, 2020
…rate"

This reverts commit f70905c.

Refresh_pillar is not needed as a pillar is gathered by the salt-minion
each time a state is run

Sees: #2444
bert-e added a commit that referenced this issue Apr 22, 2020
…into tmp/octopus/w/2.2/bugfix/GH-2444-gather-pillar-instead-of-refresh
TeddyAndrieux added a commit that referenced this issue Apr 22, 2020
…ar-instead-of-refresh' into w/2.3/bugfix/GH-2444-gather-pillar-instead-of-refresh
TeddyAndrieux added a commit that referenced this issue Apr 22, 2020
…ar-instead-of-refresh' into w/2.4/bugfix/GH-2444-gather-pillar-instead-of-refresh
bert-e added a commit that referenced this issue Apr 22, 2020
…into tmp/octopus/w/2.6/bugfix/GH-2444-gather-pillar-instead-of-refresh
bert-e added a commit that referenced this issue Apr 23, 2020
…mp/octopus/w/2.1/bugfix/GH-2444-gather-pillar-instead-of-refresh
bert-e added a commit that referenced this issue Apr 23, 2020
…into tmp/octopus/w/2.5/bugfix/GH-2444-gather-pillar-instead-of-refresh
bert-e added a commit that referenced this issue Apr 23, 2020
…' and 'q/2445/2.0/bugfix/GH-2444-gather-pillar-instead-of-refresh' into tmp/octopus/q/2.1
bert-e added a commit that referenced this issue Apr 23, 2020
…' and 'q/2445/2.1/bugfix/GH-2444-gather-pillar-instead-of-refresh' into tmp/octopus/q/2.2
bert-e added a commit that referenced this issue Apr 23, 2020
…' and 'q/2445/2.2/bugfix/GH-2444-gather-pillar-instead-of-refresh' into tmp/octopus/q/2.3
bert-e added a commit that referenced this issue Apr 23, 2020
…' and 'q/2445/2.3/bugfix/GH-2444-gather-pillar-instead-of-refresh' into tmp/octopus/q/2.4
bert-e added a commit that referenced this issue Apr 23, 2020
…' and 'q/2445/2.4/bugfix/GH-2444-gather-pillar-instead-of-refresh' into tmp/octopus/q/2.5
bert-e added a commit that referenced this issue Apr 23, 2020
…' and 'q/2445/2.5/bugfix/GH-2444-gather-pillar-instead-of-refresh' into tmp/octopus/q/2.6
@bert-e bert-e closed this as completed in 330acec Apr 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity:easy Something that requires less than a day to fix kind:bug Something isn't working topic:flakiness Some test are flaky and cause CI to do transient failing topic:lifecycle Issues related to upgrade or downgrade of MetalK8s
Projects
None yet
Development

No branches or pull requests

1 participant