Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Cp 4224 - #30

#30
Open
shriharirathod wants to merge 5 commits into
masterCacheboxInc/aerospike-server.docker:masterfrom
CP-4224CacheboxInc/aerospike-server.docker:CP-4224Copy head branch name to clipboard
Open

Cp 4224#30
shriharirathod wants to merge 5 commits into
masterCacheboxInc/aerospike-server.docker:masterfrom
CP-4224CacheboxInc/aerospike-server.docker:CP-4224Copy head branch name to clipboard

Conversation

@shriharirathod

Copy link
Copy Markdown

No description provided.

Shrihari Rathod added 3 commits December 2, 2019 18:24
    At component_stop stop heartbeating and aerospike service
    as well.

Signed-off-by: Shrihari Rathod <shrihari.rathod@primaryio.com>
Signed-off-by: Shrihari Rathod <shrihari.rathod@primaryio.com>
Signed-off-by: Shrihari Rathod <shrihari.rathod@primaryio.com>
Shrihari Rathod added 2 commits January 8, 2020 15:51
Signed-off-by: Shrihari Rathod <shrihari.rathod@primaryio.com>

@amit-pio amit-pio left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I have already reviewed this during the day time.
Looks good to me.

@shriharirathod

shriharirathod commented Mar 30, 2020

Copy link
Copy Markdown
Author

CHO was already run on PR in lite standalone mode.

@shriharirathod

Copy link
Copy Markdown
Author

@shriharirathod

Copy link
Copy Markdown
Author

@shriharirathod

Copy link
Copy Markdown
Author

API bvt run successful for PR. Logs attached for the same and reviewers added.

Comment thread src/gunicorn_config.py
def worker_int(worker):
'''
worker.log.info("Shri: worker received INT or QUIT signal")
print("Shri: worker received INT or QUIT signal")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

remove print

@kgcb kgcb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me.

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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.