Skip to content

Conversation

@Adam0Brien
Copy link
Contributor

tests if a systems shutdown timer is 2 mins and 45 seconds

@Adam0Brien Adam0Brien requested a review from travier February 17, 2023 12:41
@Adam0Brien Adam0Brien self-assigned this Feb 17, 2023
## exclusive: false
##
## Checks if default shutdown time is 2 mins an 45 seconds
##
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
##

## kola:
## exclusive: false
##
## Checks if default shutdown time is 2 mins an 45 seconds
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## Checks if default shutdown time is 2 mins an 45 seconds
# Checks if default shutdown time is 2 mins an 45 seconds

This needs to be a single #. We use double # for special kola comments, and single # for normal comments.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


# set -xeuo pipefail

# . $KOLA_EXT_DATA/commonlib.sh
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to create a folder and a symbolic link for this to work like all other tests. Look for them in other tests folders.

## Checks if default shutdown time is 2 mins an 45 seconds
##

# set -xeuo pipefail
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need this one. Your test should pass with this set.

@Adam0Brien Adam0Brien closed this Feb 20, 2023
@travier travier deleted the adams-test branch February 20, 2023 10:33
@travier
Copy link
Member

travier commented Feb 20, 2023

Moved to #2247

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