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

Kes expiration fixes #1839

Merged
merged 5 commits into from
Nov 26, 2024
Merged

Kes expiration fixes #1839

merged 5 commits into from
Nov 26, 2024

Conversation

hodlonaut
Copy link
Contributor

Description

Improvements to KES expiration information calculation/display on cntools launch, as well as pool -> list and pool -> show pages.

Where should the reviewer start?

Motivation and context

Which issue it fixes?

How has this been tested?

@hodlonaut hodlonaut requested review from rdlrt and Scitz0 November 24, 2024 12:49
Copy link
Contributor

@Scitz0 Scitz0 left a comment

Choose a reason for hiding this comment

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

I think it looks good, just added some minor comments.

Patch version should get bumped and a small changelog entry added before merged.

@@ -762,13 +762,14 @@ isPoolRegistered() {
unset p_active_epoch_no p_vrf_key_hash p_margin p_fixed_cost p_pledge p_reward_addr p_owners p_relays p_meta_url p_meta_hash p_meta_json p_pool_status
unset p_retiring_epoch p_op_cert p_op_cert_counter p_active_stake p_epoch_block_cnt p_live_stake p_live_delegators p_live_saturation
if [[ ${CNTOOLS_MODE} != "LIGHT" ]]; then
[[ -f "${POOL_FOLDER}/${1}/${POOL_REGCERT_FILENAME}" ]] && return 2 || return 1
[[ -f "${POOL_FOLDER}/${1}/${POOL_REGCERT_FILENAME}" ]] && return 2 || (rm -rf "${POOL_FOLDER}/${1}/${POOL_CURRENT_KES_START}" && return 1)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should maybe remove recursive flag as we are trying to delete a single file. So no misshaps happen if variables are unset

if [[ ! -f "${pool}/${POOL_CURRENT_KES_START}" ]]; then
continue
fi

unset pool_kes_start
Copy link
Contributor

Choose a reason for hiding this comment

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

Can remove unset of pool_kes_start as we always overwrite it on line below.

@Scitz0 Scitz0 merged commit 4853494 into alpha Nov 26, 2024
2 checks passed
@Scitz0 Scitz0 deleted the kesExpirationFixes branch November 26, 2024 22:38
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.

2 participants