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

show job cfg value of 'accelerator' (test bot PR292) #77

Closed
wants to merge 1 commit into from

Conversation

poksumdo
Copy link
Collaborator

@poksumdo poksumdo commented Feb 5, 2025

Just for testing EESSI/eessi-bot-software-layer#292

We should see the dev bot at eX3 responding when the PR is opened, and then we just close it and because there was no job there should be no comment about moving the job directories to the 🗑️

@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

No notification from the bot, and it looks like it didn't receive any event. Checking setup...

@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

Bot is listening to events from https://github.com/NorESSI/dev.nessi.no ... will change bot config and send some commands to verify it reacts.

@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

bot: show_help

@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

bot: status

@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

bot: show_help

@eessi-bot-devel-trz42
Copy link

eessi-bot-devel-trz42 bot commented Feb 5, 2025

Updates by the bot instance eX3-dev-292 (click for details)
  • received bot command show_help from poksumdo

    • expanded format: show_help
  • handling command show_help failed with message
    unknown command show_help; use bot: help for usage information

@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

bot: help

@eessi-bot-devel-trz42
Copy link

eessi-bot-devel-trz42 bot commented Feb 5, 2025

Updates by the bot instance eX3-dev-292 (click for details)
  • received bot command help from poksumdo

    • expanded format: help
  • handling command help resulted in:
    How to send commands to bot instances

    • Commands must be sent with a new comment (edits of existing comments are ignored).
    • A comment may contain multiple commands, one per line.
    • Every command begins at the start of a line and has the syntax bot: COMMAND [ARGUMENTS]*
    • Currently supported COMMANDs are: help, build, show_config, status

    For more information, see https://www.eessi.io/docs/bot

@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

Closing this to check whether the bot reports about moving no directories to the trash bin.

@poksumdo poksumdo closed this Feb 5, 2025
@poksumdo
Copy link
Collaborator Author

poksumdo commented Feb 5, 2025

Bot log contains the following

[20250205-T05:59:24] [handle_pull_request_closed_event]: PR 77: PR got closed (json value: False)
[20250205-T05:59:24] [handle_pull_request_closed_event]: PR 77: determining directories to be moved to trash bin
[20250205-T05:59:24] [handle_pull_request_closed_event]: PR 77: No job directories found; nothing to move.

The last line above is showing that the change in PR292 works. Below is the code for that

        if job_dirs == []:
            self.log(f"PR {pr.number}: No job directories found; nothing to move.")
        else:

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

Successfully merging this pull request may close these issues.

2 participants