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

chore(devservices): Bumping the version of devservices to latest #6810

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

IanWoodard
Copy link
Member

Bumping devservices version to latest which should improve reliability and fix various bugs

@IanWoodard IanWoodard requested a review from a team as a code owner January 23, 2025 19:31
Copy link

codecov bot commented Jan 23, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
2860 1 2859 11
View the top 1 failed tests by shortest run time
_integration-test.test_02_backup::test_01_backup
Stack Traces | 9.69s run time
setup_backup_restore_env_variables = None

    #x1B[0m#x1B[94mdef#x1B[39;49;00m #x1B[92mtest_01_backup#x1B[39;49;00m(setup_backup_restore_env_variables):#x1B[90m#x1B[39;49;00m
        #x1B[90m# Docker was giving me permission issues when trying to create this file and write to it even after giving read + write access#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
        #x1B[90m# to group and owner. Instead, try creating the empty file and then give everyone write access to the backup file#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
        file_path = os.path.join(os.getcwd(), #x1B[33m"#x1B[39;49;00m#x1B[33msentry#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m, #x1B[33m"#x1B[39;49;00m#x1B[33mbackup.json#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m)#x1B[90m#x1B[39;49;00m
        sentry_admin_sh = os.path.join(os.getcwd(), #x1B[33m"#x1B[39;49;00m#x1B[33msentry-admin.sh#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m)#x1B[90m#x1B[39;49;00m
        #x1B[96mopen#x1B[39;49;00m(file_path, #x1B[33m"#x1B[39;49;00m#x1B[33ma#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m, encoding=#x1B[33m"#x1B[39;49;00m#x1B[33mutf8#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m).close()#x1B[90m#x1B[39;49;00m
        os.chmod(file_path, #x1B[94m0o666#x1B[39;49;00m)#x1B[90m#x1B[39;49;00m
        #x1B[94massert#x1B[39;49;00m os.path.getsize(file_path) == #x1B[94m0#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
>       subprocess.run(#x1B[90m#x1B[39;49;00m
            [#x1B[90m#x1B[39;49;00m
                sentry_admin_sh,#x1B[90m#x1B[39;49;00m
                #x1B[33m"#x1B[39;49;00m#x1B[33mexport#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m,#x1B[90m#x1B[39;49;00m
                #x1B[33m"#x1B[39;49;00m#x1B[33mglobal#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m,#x1B[90m#x1B[39;49;00m
                #x1B[33m"#x1B[39;49;00m#x1B[33m/sentry-admin/backup.json#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m,#x1B[90m#x1B[39;49;00m
                #x1B[33m"#x1B[39;49;00m#x1B[33m--no-prompt#x1B[39;49;00m#x1B[33m"#x1B[39;49;00m,#x1B[90m#x1B[39;49;00m
            ],#x1B[90m#x1B[39;49;00m
            check=#x1B[94mTrue#x1B[39;49;00m,#x1B[90m#x1B[39;49;00m
        )#x1B[90m#x1B[39;49;00m

#x1B[1m#x1B[31m_integration-test/test_02_backup.py#x1B[0m:31: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

input = None, capture_output = False, timeout = None, check = True
popenargs = (['.../self-hosted/master/sentry-admin.sh', 'export', 'global', '/sentry-admin/backup.json', '--no-prompt'],)
kwargs = {}
process = <Popen: returncode: 1 args: ['.../_actions/getsentry/self-host...>
stdout = None, stderr = None, retcode = 1

    #x1B[0m#x1B[94mdef#x1B[39;49;00m #x1B[92mrun#x1B[39;49;00m(*popenargs,#x1B[90m#x1B[39;49;00m
            #x1B[96minput#x1B[39;49;00m=#x1B[94mNone#x1B[39;49;00m, capture_output=#x1B[94mFalse#x1B[39;49;00m, timeout=#x1B[94mNone#x1B[39;49;00m, check=#x1B[94mFalse#x1B[39;49;00m, **kwargs):#x1B[90m#x1B[39;49;00m
    #x1B[90m    #x1B[39;49;00m#x1B[33m"""Run command with arguments and return a CompletedProcess instance.#x1B[39;49;00m
    #x1B[33m#x1B[39;49;00m
    #x1B[33m    The returned instance will have attributes args, returncode, stdout and#x1B[39;49;00m
    #x1B[33m    stderr. By default, stdout and stderr are not captured, and those attributes#x1B[39;49;00m
    #x1B[33m    will be None. Pass stdout=PIPE and/or stderr=PIPE in order to capture them,#x1B[39;49;00m
    #x1B[33m    or pass capture_output=True to capture both.#x1B[39;49;00m
    #x1B[33m#x1B[39;49;00m
    #x1B[33m    If check is True and the exit code was non-zero, it raises a#x1B[39;49;00m
    #x1B[33m    CalledProcessError. The CalledProcessError object will have the return code#x1B[39;49;00m
    #x1B[33m    in the returncode attribute, and output & stderr attributes if those streams#x1B[39;49;00m
    #x1B[33m    were captured.#x1B[39;49;00m
    #x1B[33m#x1B[39;49;00m
    #x1B[33m    If timeout is given, and the process takes too long, a TimeoutExpired#x1B[39;49;00m
    #x1B[33m    exception will be raised.#x1B[39;49;00m
    #x1B[33m#x1B[39;49;00m
    #x1B[33m    There is an optional argument "input", allowing you to#x1B[39;49;00m
    #x1B[33m    pass bytes or a string to the subprocess's stdin.  If you use this argument#x1B[39;49;00m
    #x1B[33m    you may not also use the Popen constructor's "stdin" argument, as#x1B[39;49;00m
    #x1B[33m    it will be used internally.#x1B[39;49;00m
    #x1B[33m#x1B[39;49;00m
    #x1B[33m    By default, all communication is in bytes, and therefore any "input" should#x1B[39;49;00m
    #x1B[33m    be bytes, and the stdout and stderr will be bytes. If in text mode, any#x1B[39;49;00m
    #x1B[33m    "input" should be a string, and stdout and stderr will be strings decoded#x1B[39;49;00m
    #x1B[33m    according to locale encoding, or by "encoding" if set. Text mode is#x1B[39;49;00m
    #x1B[33m    triggered by setting any of text, encoding, errors or universal_newlines.#x1B[39;49;00m
    #x1B[33m#x1B[39;49;00m
    #x1B[33m    The other arguments are the same as for the Popen constructor.#x1B[39;49;00m
    #x1B[33m    """#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
        #x1B[94mif#x1B[39;49;00m #x1B[96minput#x1B[39;49;00m #x1B[95mis#x1B[39;49;00m #x1B[95mnot#x1B[39;49;00m #x1B[94mNone#x1B[39;49;00m:#x1B[90m#x1B[39;49;00m
            #x1B[94mif#x1B[39;49;00m kwargs.get(#x1B[33m'#x1B[39;49;00m#x1B[33mstdin#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m) #x1B[95mis#x1B[39;49;00m #x1B[95mnot#x1B[39;49;00m #x1B[94mNone#x1B[39;49;00m:#x1B[90m#x1B[39;49;00m
                #x1B[94mraise#x1B[39;49;00m #x1B[96mValueError#x1B[39;49;00m(#x1B[33m'#x1B[39;49;00m#x1B[33mstdin and input arguments may not both be used.#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m)#x1B[90m#x1B[39;49;00m
            kwargs[#x1B[33m'#x1B[39;49;00m#x1B[33mstdin#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m] = PIPE#x1B[90m#x1B[39;49;00m
    #x1B[90m#x1B[39;49;00m
        #x1B[94mif#x1B[39;49;00m capture_output:#x1B[90m#x1B[39;49;00m
            #x1B[94mif#x1B[39;49;00m kwargs.get(#x1B[33m'#x1B[39;49;00m#x1B[33mstdout#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m) #x1B[95mis#x1B[39;49;00m #x1B[95mnot#x1B[39;49;00m #x1B[94mNone#x1B[39;49;00m #x1B[95mor#x1B[39;49;00m kwargs.get(#x1B[33m'#x1B[39;49;00m#x1B[33mstderr#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m) #x1B[95mis#x1B[39;49;00m #x1B[95mnot#x1B[39;49;00m #x1B[94mNone#x1B[39;49;00m:#x1B[90m#x1B[39;49;00m
                #x1B[94mraise#x1B[39;49;00m #x1B[96mValueError#x1B[39;49;00m(#x1B[33m'#x1B[39;49;00m#x1B[33mstdout and stderr arguments may not be used #x1B[39;49;00m#x1B[33m'#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                                 #x1B[33m'#x1B[39;49;00m#x1B[33mwith capture_output.#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m)#x1B[90m#x1B[39;49;00m
            kwargs[#x1B[33m'#x1B[39;49;00m#x1B[33mstdout#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m] = PIPE#x1B[90m#x1B[39;49;00m
            kwargs[#x1B[33m'#x1B[39;49;00m#x1B[33mstderr#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m] = PIPE#x1B[90m#x1B[39;49;00m
    #x1B[90m#x1B[39;49;00m
        #x1B[94mwith#x1B[39;49;00m Popen(*popenargs, **kwargs) #x1B[94mas#x1B[39;49;00m process:#x1B[90m#x1B[39;49;00m
            #x1B[94mtry#x1B[39;49;00m:#x1B[90m#x1B[39;49;00m
                stdout, stderr = process.communicate(#x1B[96minput#x1B[39;49;00m, timeout=timeout)#x1B[90m#x1B[39;49;00m
            #x1B[94mexcept#x1B[39;49;00m TimeoutExpired #x1B[94mas#x1B[39;49;00m exc:#x1B[90m#x1B[39;49;00m
                process.kill()#x1B[90m#x1B[39;49;00m
                #x1B[94mif#x1B[39;49;00m _mswindows:#x1B[90m#x1B[39;49;00m
                    #x1B[90m# Windows accumulates the output in a single blocking#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                    #x1B[90m# read() call run on child threads, with the timeout#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                    #x1B[90m# being done in a join() on those threads.  communicate()#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                    #x1B[90m# _after_ kill() is required to collect that and add it#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                    #x1B[90m# to the exception.#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                    exc.stdout, exc.stderr = process.communicate()#x1B[90m#x1B[39;49;00m
                #x1B[94melse#x1B[39;49;00m:#x1B[90m#x1B[39;49;00m
                    #x1B[90m# POSIX _communicate already populated the output so#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                    #x1B[90m# far into the TimeoutExpired exception.#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                    process.wait()#x1B[90m#x1B[39;49;00m
                #x1B[94mraise#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
            #x1B[94mexcept#x1B[39;49;00m:  #x1B[90m# Including KeyboardInterrupt, communicate handled that.#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                process.kill()#x1B[90m#x1B[39;49;00m
                #x1B[90m# We don't call process.wait() as .__exit__ does that for us.#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
                #x1B[94mraise#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
            retcode = process.poll()#x1B[90m#x1B[39;49;00m
            #x1B[94mif#x1B[39;49;00m check #x1B[95mand#x1B[39;49;00m retcode:#x1B[90m#x1B[39;49;00m
>               #x1B[94mraise#x1B[39;49;00m CalledProcessError(retcode, process.args,#x1B[90m#x1B[39;49;00m
                                         output=stdout, stderr=stderr)#x1B[90m#x1B[39;49;00m
#x1B[1m#x1B[31mE               subprocess.CalledProcessError: Command '['.../self-hosted/master/sentry-admin.sh', 'export', 'global', '/sentry-admin/backup.json', '--no-prompt']' returned non-zero exit status 1.#x1B[0m

#x1B[1m#x1B[.../usr/lib/python3.12/subprocess.py#x1B[0m:571: CalledProcessError

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants