-
Notifications
You must be signed in to change notification settings - Fork 225
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
(PDB-4275) import/export configure expiration #2870
Conversation
Test PASSed |
1 similar comment
Test PASSed |
Test PASSed |
CLA signed by all contributors. |
Test FAILed |
Test PASSed |
1 similar comment
Test PASSed |
Test PASSed |
Test FAILed |
1 similar comment
Test FAILed |
src/puppetlabs/puppetdb/export.clj
Outdated
[puppetlabs.puppetdb.command :as command] | ||
[puppetlabs.puppetdb.command.constants :as command-constants] | ||
[puppetlabs.puppetdb.utils :as utils] | ||
[clj-time.format :as time-fmt] | ||
[clj-time.coerce :as time-coerce] | ||
[puppetlabs.puppetdb.schema :as pls] | ||
[schema.core :as s] | ||
[puppetlabs.i18n.core :refer [trs]])) | ||
[puppetlabs.i18n.core :refer [trs]] | ||
[flatland.ordered.map :refer [ordered-map]])) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't look like this is used, and it threw a compilation error in jenkins
src/puppetlabs/puppetdb/nodes.clj
Outdated
:- [configure-expiration-wireformat-schema] | ||
[nodes :- [nodes-wireformat-schema]] | ||
(->> nodes | ||
(filter #(false? (:expires_facts %))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
false?
is only false for the literal value false
, since expires_facts
is nil
this will end up not filtering anything right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(filter #(false? (:expires_facts %))) | |
(filter :expires_facts_updated %)) |
Test PASSed |
Adds export and import of configure-expiration commands via the nodes queries. This will only export nodes that have an explicit factset expiration status set.
Test PASSed |
Adds export and import of configure-expiration commands via the nodes
queries