Skip to content

list reserved date and allow filtering by reservered for background job list command - #38367

Closed
icewind1991 wants to merge 1 commit into
masterfrom
background-job-list-reserved
Closed

list reserved date and allow filtering by reservered for background job list command#38367
icewind1991 wants to merge 1 commit into
masterfrom
background-job-list-reserved

Conversation

@icewind1991

@icewind1991icewind1991 commented May 19, 2023

Copy link
Copy Markdown
Member
  • Add "reserved at" to the output of occ background-job:list
  • Add option to only list reserved jobs with occ background-job:list --reserved

Should be useful when investigating cron issues

@icewind1991icewind1991 added the 3. to review Waiting for reviews label May 19, 2023
@icewind1991icewind1991 added this to the Nextcloud 28 milestone May 19, 2023
@icewind1991
icewind1991 requested review from a team, ArtificialOwl, come-nc and nfebe and removed request for a teamMay 19, 2023 13:19
github-advanced-security[bot]
github-advanced-securityAI found potential problems May 19, 2023
View reviewed changes
Comment threadlib/private/BackgroundJob/JobList.php Fixed
come-nc
come-nc reviewed May 22, 2023
View reviewed changes
Comment threadlib/private/BackgroundJob/Job.php Outdated
Comment threadlib/public/BackgroundJob/IJob.php Outdated
Comment threadlib/public/BackgroundJob/Job.php Outdated
Comment threadlib/private/BackgroundJob/JobList.php
@icewind1991icewind1991 changed the title list reserved data and allow filtering by reservered for background job list commandlist reserved date and allow filtering by reservered for background job list commandMay 22, 2023
@icewind1991
list reserved data and allow filtering by reservered for background j…
bd5905a
…ob list command
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991
icewind1991force-pushed the background-job-list-reserved branch from 968927a to bd5905aCompareJune 9, 2023 17:37
come-nc
come-nc approved these changes Jun 12, 2023
View reviewed changes
Comment threadlib/public/BackgroundJob/IJob.php
/**
* Get the timestamp when the job was reserved, or 0 if the job is not currently reserved
*
* @return int

@come-nccome-ncJun 12, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
* @return int
* @since 28.0.0

@skjnldsvskjnldsv mentioned this pull request Nov 1, 2023
Merged
This was referenced Nov 6, 2023
Merged
Merged
This was referenced Nov 14, 2023
Merged
Merged
@blizzzblizzz modified the milestones: Nextcloud 28, Nextcloud 29Nov 23, 2023
@skjnldsvskjnldsv added 2. developing Work in progress technical debt 🧱 🤔🚀 and removed 3. to review Waiting for reviews labels Feb 24, 2024
This was referenced Mar 12, 2024
Merged
Merged
Merged
This was referenced Mar 20, 2024
Merged
Merged
@skjnldsvskjnldsv mentioned this pull request Mar 28, 2024
Merged
81 tasks
@skjnldsvskjnldsv removed this from the Nextcloud 29 milestone Mar 28, 2024
@skjnldsvskjnldsv added this to the Nextcloud 30 milestone Mar 28, 2024
This was referenced Jul 30, 2024
Merged
Merged
This was referenced Aug 5, 2024
Merged
Merged
@skjnldsvskjnldsv mentioned this pull request Aug 13, 2024
Merged
@skjnldsvskjnldsv removed this from the Nextcloud 30 milestone Aug 14, 2024
@skjnldsv
skjnldsv deleted the background-job-list-reserved branch December 19, 2024 18:04
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@come-nccome-nccome-nc approved these changes
@ArtificialOwlArtificialOwlAwaiting requested review from ArtificialOwlArtificialOwl is a code owner automatically assigned from nextcloud/server-backend
@nfebenfebeAwaiting requested review from nfebenfebe is a code owner automatically assigned from nextcloud/server-backend

Assignees

No one assigned

Labels

2. developingWork in progresstechnical debt🧱 🤔🚀

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@icewind1991@github-advanced-security@come-nc@blizzz@skjnldsv