Skip to content

Conversation

@yambenari24
Copy link
Contributor

@yambenari24 yambenari24 commented Aug 17, 2025

Pull Request Checklist

Please complete the following before submitting:

General notes -

  • I have tested the changes locally.
  • I have written unit tests where applicable.
  • I have updated documentation where needed.
  • I have added comments to complex code.
  • This PR follows the coding style guidelines.
  • I have updated release notes with new feature

New Kaltura Types

  • I have created new clients
  • I have notified related apps - KMCNG / KMS / EP .... about new clients

New Kaltura Services / Actions

  • I have added a deployment script

Questions

  1. What is the purpose of this PR?

    • Enter your answer here
  2. Does this change affect production code or infrastructure?

    • Yes
    • No
  3. If yes, what is the rollback plan?

    • Enter your answer here

@github-actions
Copy link

@github-copilot suggest

@github-actions
Copy link

@github-copilot suggest

@github-actions
Copy link

@github-copilot suggest

if ($argc < 2)
{
echo UNIX_LINE_END . ' ---- Delete Kuser From Elastic ---- ' . UNIX_LINE_END;
echo ' Execute: php ' . $argv[0] . ' [ /path/to/kuser_id.log || kuserId_1,kuserId_2,kuserId_3,.. ] [realrun / dryrun]' . UNIX_LINE_END;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did it meant - path/to/userIdFile ?

How will the values be in the file? one ID per line or CSV?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

  1. When using a file path (e.g., /path/to/kuser_id.log), the file should contain one Kuser ID per line.
  2. When providing IDs directly in the command, they should be passed as a comma-separated list

@github-actions
Copy link

@github-copilot suggest

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants