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.

Sign up for GitHub

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

Jump to bottom

Fix kubernetes provider synchronization issues #802

Merged
merged 5 commits into from
Mar 20, 2023
Merged

Fix kubernetes provider synchronization issues #802

merged 5 commits into from
Mar 20, 2023

Commits on Mar 18, 2023

  1. Remove unused fn argument 

    fn: KubeClient::createResource
arg: waitReady
    @serban300
    serban300 committed Mar 18, 2023
    Configuration menu
    Copy the full SHA
    3cdc2b7 View commit details
    Browse the repository at this point in the history

  2. Fix KubeClient::wait_transfer_container() 

    - We were assessing if the transfer-files-container is ready only by
checking if it's state is "running". While this is necessary, it is not
enough. We need to make sure that it finished initializing (downloading
coreutils). We do this by looking for the "waiting for tar to finish"
line in the logs
- Define a reusable method for retrying actions.
    @serban300
    serban300 committed Mar 18, 2023
    Configuration menu
    Copy the full SHA
    b3764fb View commit details
    Browse the repository at this point in the history

  3. Fix k8s provider setupChainSpec() 

    - Wait for the chain to start before trying to copy the chain spec file
- Use wait_log() for getChainSpecRaw()
    @serban300
    serban300 committed Mar 18, 2023
    Configuration menu
    Copy the full SHA
    64749a9 View commit details
    Browse the repository at this point in the history

  4. Fix wget failures 

    I stumbled upon a lot of failures when downloading coreutils, after curl
had been downloaded successfully from the same server. Not sure why.
Might be an issue with the wget used in the container. Anyway, not using
https seems to fix the issue.
    @serban300
    serban300 committed Mar 18, 2023
    Configuration menu
    Copy the full SHA
    abca336 View commit details
    Browse the repository at this point in the history

  5. Run prettier

    @serban300
    serban300 committed Mar 18, 2023
    Configuration menu
    Copy the full SHA
    e351e95 View commit details
    Browse the repository at this point in the history