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

Add ability to just unjoin domain #98

Merged
merged 2 commits into from
Jul 19, 2016

Conversation

propyless
Copy link

No description provided.

@propyless
Copy link
Author

lemme just do some tests on live staging servers

@propyless
Copy link
Author

puppet agent -t --environment test_vas
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Info: Caching catalog for themostawesomenode.awesome.com
Info: Applying configuration version 'this_cataogue_was_generated_by_the_best_server'
Notice: /Stage[main]/Vas/Exec[vas_unjoin]/returns: executed successfully
Notice: Finished catalog run in 14.18 seconds
[root@themostawesomenode ~]# /opt/quest/bin/vastool status
Host:   <themostawesomenode.awesome.com, Linux x86_64>
Date:   <Fri Apr 29 10:38:09 CEST 2016>
QAS:    <4.1.0>
Domain: <N/A>
CRITICAL: 106 /etc/opt/quest/vas/vas.conf missing default_realm setting, is QAS joined?
Result: <Critical test(s) failed> (0 seconds)(v0.7.6)

@propyless
Copy link
Author

@Phil-Friderici ping!

@Phil-Friderici
Copy link
Contributor

@propyless looks like the code has much more changes than the tests. How come ?

@Phil-Friderici
Copy link
Contributor

@propyless never mind! The diff doesn't show the changed indenting very well.

@propyless
Copy link
Author

:)

Mvh
Takeshi

On 4 maj 2016, at 09:32, Phil Friderici notifications@github.com wrote:

@propyless never mind! The diff doesn't show the changed indenting very well.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@propyless
Copy link
Author

I have identified some issues with the PR which I will resolve...

@Phil-Friderici
Copy link
Contributor

Phil-Friderici commented May 10, 2016

@propyless you'll need to add this to your Puppet code:

# lint:ignore:only_variable_string

like here: [redacted]

@propyless
Copy link
Author

propyless commented May 10, 2016

You are linking to a private repository to which I do not have access ;)
Also it was a mistake in the code on my part so I've fixed that now

@Phil-Friderici
Copy link
Contributor

Phil-Friderici commented May 10, 2016

oh! sorry, but I see the specs are passing again 👍

Both files should exist, not either which.
@propyless
Copy link
Author

I also added a bonus commit that fixes the logic in the if statement for vas migration part.

@Phil-Friderici Phil-Friderici merged commit 72b8f35 into Ericsson:master Jul 19, 2016
@Phil-Friderici
Copy link
Contributor

got positive feedback from our VAS specialist :)

@Phil-Friderici
Copy link
Contributor

@propyless Thanks a lot! Released in v0.18.0
Enjoy your vacation days ☀️ have phun and a good start afterwards 🍀

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