Please use '_stored' instead of 'state'

Bug #1888636 reported by John Lenton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
charm-k8s-jenkins-agent
Fix Released
Medium
Unassigned

Bug Description

We've found that 'state' is ambiguous for StoredState, please if possible use something like '_stored' for it.

Also note the only use you have of stored state right now is saving the previous pod spec, which AWUI is unnecessary as setting an unchanged pod spec does not cause a restart.

Tom Haddon (mthaddon)
Changed in charm-k8s-jenkins-agent:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Tom Haddon (mthaddon) wrote :

This has now been fixed.

Changed in charm-k8s-jenkins-agent:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.